diff options
| author | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-12 00:39:15 +0000 |
|---|---|---|
| committer | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-12 00:39:15 +0000 |
| commit | c002879632b0e46b11e91e5595a83652f7cafd3c (patch) | |
| tree | 87477d9e6b972292fbb98484ec3721f084ab41c4 /base/base.gypi | |
| parent | 62ed4d36e4357deaf863711bdfd1f00b367c426a (diff) | |
| download | chromium_src-c002879632b0e46b11e91e5595a83652f7cafd3c.zip chromium_src-c002879632b0e46b11e91e5595a83652f7cafd3c.tar.gz chromium_src-c002879632b0e46b11e91e5595a83652f7cafd3c.tar.bz2 | |
Mac: Other approach for IPCing child task_ts.
Also move mach_ipc_mac to base, where it's now used.
Based on http://www.foldr.org/~michaelw/log/2009/03/13/ , but uses a named connection instead. Do the IPC right after fork-time, so that the sandbox is not yet in effect.
See the codereview comments for a benchmark that proves that this shouldn't be expensive, and for pros and cons for using a named connection vs temporarily switching out the bootstrap port.
Works for worker processes too and seems more reliable in general.
Measured perf impact in http://src.chromium.org/viewvc/chrome?view=rev&revision=35888 , it's negligible.
BUG=13156
TEST=(requires that one enables the task manager in browser.cc)
1.) Open one tab that plays a youtube video
2.) Open a second and visit http://www.whatwg.org/demos/workers/primes/page.html
3.) Install e.g. the gmail checker extension
4.) Open the task manager
It should report metrics for
* one browser process
* two renderer processes
* one plugin process
* one worker process
* one extension process
Check that %cpu etc more or less match what Activity Monitor displays if you filter for "Chromium".
Also choose "Open all bookmarks" on the bookmarks bar with the task manager open and check that metrics for all tabs appear immediately.
Review URL: http://codereview.chromium.org/549002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35977 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/base.gypi')
| -rw-r--r-- | base/base.gypi | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/base/base.gypi b/base/base.gypi index fc7c811..f7584c4 100644 --- a/base/base.gypi +++ b/base/base.gypi @@ -103,6 +103,8 @@ 'logging.h', 'mac_util.h', 'mac_util.mm', + 'mach_ipc_mac.h', + 'mach_ipc_mac.mm', 'memory_debug.cc', 'memory_debug.h', 'message_loop.cc', |