| 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/broker_process_dispatcher.h" | 5 #include "content/ppapi_plugin/broker_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 broker process. | 13 // How long we wait before releasing the broker process. |
| 14 const int kBrokerReleaseTimeMs = 30 * 1000; // 30 seconds. | 14 const int kBrokerReleaseTimeSeconds = 30; |
| 15 | 15 |
| 16 } // namespace | 16 } // namespace |
| 17 | 17 |
| 18 BrokerProcessDispatcher::BrokerProcessDispatcher( | 18 BrokerProcessDispatcher::BrokerProcessDispatcher( |
| 19 base::ProcessHandle remote_process_handle, | 19 base::ProcessHandle remote_process_handle, |
| 20 PP_ConnectInstance_Func connect_instance) | 20 PP_ConnectInstance_Func connect_instance) |
| 21 : ppapi::proxy::BrokerSideDispatcher(remote_process_handle, | 21 : ppapi::proxy::BrokerSideDispatcher(remote_process_handle, |
| 22 connect_instance) { | 22 connect_instance) { |
| 23 ChildProcess::current()->AddRefProcess(); | 23 ChildProcess::current()->AddRefProcess(); |
| 24 } | 24 } |
| 25 | 25 |
| 26 BrokerProcessDispatcher::~BrokerProcessDispatcher() { | 26 BrokerProcessDispatcher::~BrokerProcessDispatcher() { |
| 27 DVLOG(1) << "BrokerProcessDispatcher::~BrokerProcessDispatcher()"; | 27 DVLOG(1) << "BrokerProcessDispatcher::~BrokerProcessDispatcher()"; |
| 28 // Don't free the process right away. This timer allows the child process | 28 // Don't free the process right away. This timer allows the child process |
| 29 // to be re-used if the user rapidly goes to a new page that requires this | 29 // to be re-used if the user rapidly goes to a new page that requires this |
| 30 // plugin. This is the case for common plugins where they may be used on a | 30 // plugin. This is the case for common plugins where they may be used on a |
| 31 // source and destination page of a navigation. We don't want to tear down | 31 // source and destination page of a navigation. We don't want to tear down |
| 32 // and re-start processes each time in these cases. | 32 // and re-start processes each time in these cases. |
| 33 MessageLoop::current()->PostDelayedTask( | 33 MessageLoop::current()->PostDelayedTask( |
| 34 FROM_HERE, | 34 FROM_HERE, |
| 35 base::Bind(&ChildProcess::ReleaseProcess, | 35 base::Bind(&ChildProcess::ReleaseProcess, |
| 36 base::Unretained(ChildProcess::current())), | 36 base::Unretained(ChildProcess::current())), |
| 37 kBrokerReleaseTimeMs); | 37 base::TimeDelta::FromSeconds(kBrokerReleaseTimeSeconds)); |
| 38 } | 38 } |
| OLD | NEW |