summaryrefslogtreecommitdiffstats
path: root/ppapi/proxy/ppp_instance_private_proxy_unittest.cc
diff options
context:
space:
mode:
authordmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-03-12 00:41:08 +0000
committerdmichael@chromium.org <dmichael@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-03-12 00:41:08 +0000
commit6fb8da6d97a3bbdb0df022ea201bf91aa36a001b (patch)
tree26d3b38ccb77f4fdb5dc2538c67e42703295bde5 /ppapi/proxy/ppp_instance_private_proxy_unittest.cc
parent5e93c47a282ed564e6a56af10f4097599fe41def (diff)
downloadchromium_src-6fb8da6d97a3bbdb0df022ea201bf91aa36a001b.zip
chromium_src-6fb8da6d97a3bbdb0df022ea201bf91aa36a001b.tar.gz
chromium_src-6fb8da6d97a3bbdb0df022ea201bf91aa36a001b.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: 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 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=187340 Review URL: https://chromiumcodereview.appspot.com/12378050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187427 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.cc8
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());