| OLD | NEW | 
|---|
| 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be | 
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. | 
| 4 | 4 | 
| 5 #include "content/ppapi_plugin/plugin_process_dispatcher.h" | 5 #include "content/ppapi_plugin/plugin_process_dispatcher.h" | 
| 6 | 6 | 
| 7 #include "base/bind.h" | 7 #include "base/bind.h" | 
| 8 #include "base/bind_helpers.h" | 8 #include "base/bind_helpers.h" | 
| 9 #include "content/common/child_process.h" | 9 #include "content/common/child_process.h" | 
| 10 | 10 | 
| 11 namespace { | 11 namespace { | 
| 12 | 12 | 
| 13 // How long we wait before releasing the plugin process. | 13 // How long we wait before releasing the plugin process. | 
| 14 const int kPluginReleaseTimeMs = 30 * 1000;  // 30 seconds. | 14 const int kPluginReleaseTimeSeconds = 30; | 
| 15 | 15 | 
| 16 }  // namespace | 16 }  // namespace | 
| 17 | 17 | 
| 18 PluginProcessDispatcher::PluginProcessDispatcher( | 18 PluginProcessDispatcher::PluginProcessDispatcher( | 
| 19     base::ProcessHandle remote_process_handle, | 19     base::ProcessHandle remote_process_handle, | 
| 20     GetInterfaceFunc get_interface) | 20     GetInterfaceFunc get_interface) | 
| 21     : ppapi::proxy::PluginDispatcher(remote_process_handle, get_interface) { | 21     : ppapi::proxy::PluginDispatcher(remote_process_handle, get_interface) { | 
| 22   ChildProcess::current()->AddRefProcess(); | 22   ChildProcess::current()->AddRefProcess(); | 
| 23 } | 23 } | 
| 24 | 24 | 
| 25 PluginProcessDispatcher::~PluginProcessDispatcher() { | 25 PluginProcessDispatcher::~PluginProcessDispatcher() { | 
| 26   // Don't free the process right away. This timer allows the child process | 26   // Don't free the process right away. This timer allows the child process | 
| 27   // to be re-used if the user rapidly goes to a new page that requires this | 27   // to be re-used if the user rapidly goes to a new page that requires this | 
| 28   // plugin. This is the case for common plugins where they may be used on a | 28   // plugin. This is the case for common plugins where they may be used on a | 
| 29   // source and destination page of a navigation. We don't want to tear down | 29   // source and destination page of a navigation. We don't want to tear down | 
| 30   // and re-start processes each time in these cases. | 30   // and re-start processes each time in these cases. | 
| 31   MessageLoop::current()->PostDelayedTask( | 31   MessageLoop::current()->PostDelayedTask( | 
| 32       FROM_HERE, | 32       FROM_HERE, | 
| 33       base::Bind(&ChildProcess::ReleaseProcess, | 33       base::Bind(&ChildProcess::ReleaseProcess, | 
| 34                  base::Unretained(ChildProcess::current())), | 34                  base::Unretained(ChildProcess::current())), | 
| 35       kPluginReleaseTimeMs); | 35       base::TimeDelta::FromSeconds(kPluginReleaseTimeSeconds)); | 
| 36 } | 36 } | 
| OLD | NEW | 
|---|