diff options
author | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-30 17:08:13 +0000 |
---|---|---|
committer | dmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-30 17:08:13 +0000 |
commit | b73c659d632b24e11e2316a0285b5475ccffc3e2 (patch) | |
tree | 230cb5a0b468612ffa41e9c4af9052f32fd3cf52 /ppapi/proxy/ppp_instance_private_proxy_unittest.cc | |
parent | 29190fda701edd26bf862b3d3b91bab66fe2c88b (diff) | |
download | chromium_src-b73c659d632b24e11e2316a0285b5475ccffc3e2.zip chromium_src-b73c659d632b24e11e2316a0285b5475ccffc3e2.tar.gz chromium_src-b73c659d632b24e11e2316a0285b5475ccffc3e2.tar.bz2 |
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: r186925
Reverted: r186939 due to build errors
Committed: r187340
Committed: r187427
Reverted: r187668 due to a failing check in Canary, which was fixed here: r187681
Committed: r189518
Reverted: r189682, due to regression in Kraken (see crbug.com/222741)
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=191420
Reverted: r191441, due to new failing PDFResource unit test.
Review URL: https://chromiumcodereview.appspot.com/12378050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@191510 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/proxy/ppp_instance_private_proxy_unittest.cc')
-rw-r--r-- | ppapi/proxy/ppp_instance_private_proxy_unittest.cc | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/ppapi/proxy/ppp_instance_private_proxy_unittest.cc b/ppapi/proxy/ppp_instance_private_proxy_unittest.cc index df1af0c..317f4d4 100644 --- a/ppapi/proxy/ppp_instance_private_proxy_unittest.cc +++ b/ppapi/proxy/ppp_instance_private_proxy_unittest.cc @@ -147,15 +147,7 @@ class PPP_Instance_Private_ProxyTest : public TwoWayTest { } // namespace -// TODO(raymes): This #ifdef is only here because we check the state of the -// plugin globals on the main thread, rather than the plugin thread which causes -// the thread checker to fail. Once ENABLE_PEPPER_THREADING is the default, -// this will be safe to do anyway, so we can remove this. -#ifdef ENABLE_PEPPER_THREADING TEST_F(PPP_Instance_Private_ProxyTest, PPPInstancePrivate) { -#else -TEST_F(PPP_Instance_Private_ProxyTest, DISABLED_PPPInstancePrivate) { -#endif // This test controls its own instance; we can't use the one that // PluginProxyTestHarness provides. ASSERT_NE(kInstance, pp_instance()); |