diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-27 20:45:59 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-03-27 20:45:59 +0000 |
| commit | 51d70e07bd5d991420bc0f14ff05f1a9b72d02db (patch) | |
| tree | 718c0b24680c1b14eb4570403cc15b8d09b4002d /chrome/worker/worker_main.cc | |
| parent | 70ac249f04337f0040ad63a72ced111d87fdc73a (diff) | |
| download | chromium_src-51d70e07bd5d991420bc0f14ff05f1a9b72d02db.zip chromium_src-51d70e07bd5d991420bc0f14ff05f1a9b72d02db.tar.gz chromium_src-51d70e07bd5d991420bc0f14ff05f1a9b72d02db.tar.bz2 | |
Refactor plugin process code which checks with the browser process before shutdown, to avoid races in which the browser process thinks the process is fine to use while it's shutting down. I also removed PluginProcess/WorkerProcess since they didn't have any code in them now.
I removed the plugin process code which waits 10 seconds before shutting itself down. That was a premature optimization, since testing with/without this didn't show any difference (see http://www/~jabdelmalek/chrome/test/plugins/processes.html). In both cases, the plugin on a page would get recreated in less than 100ms, even with reusing or starting a plugin process from scratch. We already spawn new renderer processes on back and forth if it's a different origin, and the plugin will be in the cache anyways.
Review URL: http://codereview.chromium.org/53091
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12703 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/worker/worker_main.cc')
| -rw-r--r-- | chrome/worker/worker_main.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/chrome/worker/worker_main.cc b/chrome/worker/worker_main.cc index 1b973c3..ad41a8b 100644 --- a/chrome/worker/worker_main.cc +++ b/chrome/worker/worker_main.cc @@ -6,12 +6,13 @@ #include "base/message_loop.h" #include "base/string_util.h" #include "base/system_monitor.h" +#include "chrome/common/child_process.h" #include "chrome/common/chrome_constants.h" #include "chrome/common/chrome_switches.h" #include "chrome/common/logging_chrome.h" #include "chrome/common/main_function_params.h" #include "chrome/common/win_util.h" -#include "chrome/worker/worker_process.h" +#include "chrome/worker/worker_thread.h" #if defined(OS_WIN) #include "chrome/common/sandbox_init_wrapper.h" @@ -30,7 +31,7 @@ int WorkerMain(const MainFunctionParams& parameters) { // Initialize the SystemMonitor base::SystemMonitor::Start(); - WorkerProcess worker_process; + ChildProcess worker_process(new WorkerThread()); #if defined(OS_WIN) sandbox::TargetServices* target_services = parameters.sandbox_info_.TargetServices(); |