Re: [chromium-dev] Process Model Extension API Proposal

2010-01-16 Thread Darin Fisher
On Fri, Jan 15, 2010 at 1:27 PM, Charlie Reis cr...@chromium.org wrote: On Fri, Jan 15, 2010 at 12:04 PM, Rafael Weinstein rafa...@chromium.orgwrote: LGTM also. One thought: I've seen a few requests on the crx-dev mailing list for API capabilities for both finding out when a process has

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-15 Thread Rafael Weinstein
LGTM also. One thought: I've seen a few requests on the crx-dev mailing list for API capabilities for both finding out when a process has crashed and also when it has become unresponsive. I'm unfamiliar with our code that checks for a hung renderer. Would the process.cpu value be a proxy for

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-15 Thread Charlie Reis
On Fri, Jan 15, 2010 at 12:04 PM, Rafael Weinstein rafa...@chromium.orgwrote: LGTM also. One thought: I've seen a few requests on the crx-dev mailing list for API capabilities for both finding out when a process has crashed and also when it has become unresponsive. I'm unfamiliar with our

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-15 Thread Rafael Weinstein
On Fri, Jan 15, 2010 at 1:27 PM, Charlie Reis cr...@chromium.org wrote: On Fri, Jan 15, 2010 at 12:04 PM, Rafael Weinstein rafa...@chromium.org wrote: LGTM also. One thought: I've seen a few requests on the crx-dev mailing list for API capabilities for both finding out when a process has

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-15 Thread Charlie Reis
On Fri, Jan 15, 2010 at 1:33 PM, Rafael Weinstein rafa...@chromium.orgwrote: On Fri, Jan 15, 2010 at 1:27 PM, Charlie Reis cr...@chromium.org wrote: On Fri, Jan 15, 2010 at 12:04 PM, Rafael Weinstein rafa...@chromium.org wrote: LGTM also. One thought: I've seen a few requests

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-13 Thread Charlie Reis
Thanks. I've updated the proposal doc based on suggestions from this discussion, including a global onUpdated event, notes about only providing cross-platform statistics, and adding a tabs array to each Process for the tabs associated with it (at least for renderer processes).

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-13 Thread Erik Kay
On Wed, Jan 13, 2010 at 1:21 PM, Charlie Reis cr...@chromium.org wrote: Thanks. I've updated the proposal doc based on suggestions from this discussion, including a global onUpdated event, notes about only providing cross-platform statistics, and adding a tabs array to each Process for the

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-12 Thread Matt Perry
Would it be possible to implement the same functionality using an NPAPI plugin? Extensions are allowed to bundle native plugins and interact with them from script. On Tue, Jan 12, 2010 at 10:48 AM, Charlie Reis cr...@chromium.org wrote: Hi folks-- I've put together a proposal for a Chromium

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-12 Thread James Robinson
Getting this information in a cross-platform way is a huge pain (do we even do it properly for mac yet?), so it seems like a decent idea to let extensions reuse the work done rather than reimplementing everything. I'd suggest having the browser fire some sort of update event to the extension

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-12 Thread Charlie Reis
On Tue, Jan 12, 2010 at 12:44 PM, James Robinson jam...@google.com wrote: Getting this information in a cross-platform way is a huge pain (do we even do it properly for mac yet?), so it seems like a decent idea to let extensions reuse the work done rather than reimplementing everything. I

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-12 Thread Nico Weber
(re-sending from chromium address) On Tue, Jan 12, 2010 at 12:44 PM, James Robinson jam...@google.com wrote: Getting this information in a cross-platform way is a huge pain (do we even do it properly for mac yet?), Yes, since yesterday (at least most of it). Note that some of the memory

Re: [chromium-dev] Process Model Extension API Proposal

2010-01-12 Thread Erik Kay
LGTM. I agree that you should look into the broad (global, not per-process) onUpdated event. For any active monitoring extension (vs. static display), I would wager that this would result in a more efficient implementation than having them poll. I also agree with Aaron's answers to your open