diff options
author | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-11 19:58:09 +0000 |
---|---|---|
committer | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-11 19:58:09 +0000 |
commit | a65fe6ed7d949e55e9fca008ec7805e4f1f19a23 (patch) | |
tree | c3f575a3c9911f48a0c145806272ab225ae6a0d3 /ppapi/proxy/ppapi_proxy_test.cc | |
parent | 5102497f1d1dc0cc29967f66ac103185e279b5c6 (diff) | |
download | chromium_src-a65fe6ed7d949e55e9fca008ec7805e4f1f19a23.zip chromium_src-a65fe6ed7d949e55e9fca008ec7805e4f1f19a23.tar.gz chromium_src-a65fe6ed7d949e55e9fca008ec7805e4f1f19a23.tar.bz2 |
Revert 187340 "PPAPI: Remove threading options; it's always on"
> PPAPI: Remove threading options; it's always on
>
> This also re-enables thread checking for the host side resource and var trackers. Before, checking was disabled everywhere.
>
> BUG=159240,92909
>
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=186925
> Reverted: https://src.chromium.org/viewvc/chrome?view=rev&revision=186939 due to build errors
>
> Review URL: https://chromiumcodereview.appspot.com/12378050
TBR=dmichael@chromium.org
Review URL: https://codereview.chromium.org/12476028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187346 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/proxy/ppapi_proxy_test.cc')
-rw-r--r-- | ppapi/proxy/ppapi_proxy_test.cc | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/ppapi/proxy/ppapi_proxy_test.cc b/ppapi/proxy/ppapi_proxy_test.cc index 9034d8f..00947e3 100644 --- a/ppapi/proxy/ppapi_proxy_test.cc +++ b/ppapi/proxy/ppapi_proxy_test.cc @@ -171,8 +171,6 @@ Dispatcher* PluginProxyTestHarness::GetDispatcher() { void PluginProxyTestHarness::SetUpHarness() { // These must be first since the dispatcher set-up uses them. CreatePluginGlobals(); - // Some of the methods called during set-up check that the lock is held. - ProxyAutoLock lock; resource_tracker().DidCreateInstance(pp_instance()); @@ -198,8 +196,6 @@ void PluginProxyTestHarness::SetUpHarnessWithChannel( bool is_client) { // These must be first since the dispatcher set-up uses them. CreatePluginGlobals(); - // Some of the methods called during set-up check that the lock is held. - ProxyAutoLock lock; resource_tracker().DidCreateInstance(pp_instance()); plugin_delegate_mock_.Init(ipc_message_loop, shutdown_event); @@ -218,15 +214,10 @@ void PluginProxyTestHarness::SetUpHarnessWithChannel( } void PluginProxyTestHarness::TearDownHarness() { - { - // Some of the methods called during tear-down check that the lock is held. - ProxyAutoLock lock; - - plugin_dispatcher_->DidDestroyInstance(pp_instance()); - plugin_dispatcher_.reset(); + plugin_dispatcher_->DidDestroyInstance(pp_instance()); + plugin_dispatcher_.reset(); - resource_tracker().DidDeleteInstance(pp_instance()); - } + resource_tracker().DidDeleteInstance(pp_instance()); plugin_globals_.reset(); } |