diff options
| author | ddorwin@chromium.org <ddorwin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-26 22:27:41 +0000 |
|---|---|---|
| committer | ddorwin@chromium.org <ddorwin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-26 22:27:41 +0000 |
| commit | 1977348dd202ea5bc716f44be23972520c434ed9 (patch) | |
| tree | e800c4b58354769a7752c384508c85a589aff5ba /content/ppapi_plugin | |
| parent | 4cf30858ab646a9e315fb9515a06c5d58c6da575 (diff) | |
| download | chromium_src-1977348dd202ea5bc716f44be23972520c434ed9.zip chromium_src-1977348dd202ea5bc716f44be23972520c434ed9.tar.gz chromium_src-1977348dd202ea5bc716f44be23972520c434ed9.tar.bz2 | |
Added logging useful for understanding Pepper broker lifetime.
Also, added support for VLOG to Pepper OOP plugin and Pepper broker processes by forwarding --vmodule to these processes.
BUG=none
TEST=Launch a dbg build of Chrome with --vmodule=ppapi_plugin_process_host=1,broker_process_dispatcher=1,ppapi_broker_main=1,broker_dispatcher=1,proxy_channel=1", visit a page that uses the broker, and navigate away from that page. The new log statements should be displayed.
Review URL: http://codereview.chromium.org/8347009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107451 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/ppapi_plugin')
| -rw-r--r-- | content/ppapi_plugin/broker_process_dispatcher.cc | 12 | ||||
| -rw-r--r-- | content/ppapi_plugin/ppapi_broker_main.cc | 1 |
2 files changed, 8 insertions, 5 deletions
diff --git a/content/ppapi_plugin/broker_process_dispatcher.cc b/content/ppapi_plugin/broker_process_dispatcher.cc index b8c31d3..26a7100 100644 --- a/content/ppapi_plugin/broker_process_dispatcher.cc +++ b/content/ppapi_plugin/broker_process_dispatcher.cc @@ -8,15 +8,16 @@ namespace { -class PluginReleaseTask : public Task { +class BrokerReleaseTask : public Task { public: void Run() { + DVLOG(1) << "BrokerReleaseTask::Run()"; ChildProcess::current()->ReleaseProcess(); } }; -// How long we wait before releasing the plugin process. -const int kPluginReleaseTimeMs = 30 * 1000; // 30 seconds. +// How long we wait before releasing the broker process. +const int kBrokerReleaseTimeMs = 30 * 1000; // 30 seconds. } // namespace @@ -29,11 +30,12 @@ BrokerProcessDispatcher::BrokerProcessDispatcher( } BrokerProcessDispatcher::~BrokerProcessDispatcher() { + DVLOG(1) << "BrokerProcessDispatcher::~BrokerProcessDispatcher()"; // Don't free the process right away. This timer allows the child process // to be re-used if the user rapidly goes to a new page that requires this // plugin. This is the case for common plugins where they may be used on a // source and destination page of a navigation. We don't want to tear down // and re-start processes each time in these cases. - MessageLoop::current()->PostDelayedTask(FROM_HERE, new PluginReleaseTask(), - kPluginReleaseTimeMs); + MessageLoop::current()->PostDelayedTask(FROM_HERE, new BrokerReleaseTask(), + kBrokerReleaseTimeMs); } diff --git a/content/ppapi_plugin/ppapi_broker_main.cc b/content/ppapi_plugin/ppapi_broker_main.cc index 17cd7b7..8789a61 100644 --- a/content/ppapi_plugin/ppapi_broker_main.cc +++ b/content/ppapi_plugin/ppapi_broker_main.cc @@ -24,5 +24,6 @@ int PpapiBrokerMain(const MainFunctionParams& parameters) { ppapi_broker_process.set_main_thread(new PpapiThread(true)); // Broker. main_message_loop.Run(); + DVLOG(1) << "PpapiBrokerMain exiting"; return 0; } |