Chromium Code Reviews| Index: chrome/test/gpu/gpu_feature_browsertest.cc |
| diff --git a/chrome/test/gpu/gpu_feature_browsertest.cc b/chrome/test/gpu/gpu_feature_browsertest.cc |
| index 8a6af169af580e2cf72bbe7149b4cf9c85a21406..3c63bc7cebdd78668383d7934106b44052625033 100644 |
| --- a/chrome/test/gpu/gpu_feature_browsertest.cc |
| +++ b/chrome/test/gpu/gpu_feature_browsertest.cc |
| @@ -185,11 +185,15 @@ IN_PROC_BROWSER_TEST_F(GpuFeatureTest, MAYBE_AcceleratedCompositingAllowed) { |
| RunEventTest(url, kSwapBuffersEvent, true); |
| } |
| -class AcceleratedCompositingBlockedTest : public GpuFeatureTest { |
| - public: |
| - virtual void SetUpInProcessBrowserTestFixture() OVERRIDE { |
| - GpuFeatureTest::SetUpInProcessBrowserTestFixture(); |
| - const std::string json_blacklist = |
| +#if defined(USE_AURA) |
| +// Compositing is always on for Aura. |
| +#define MAYBE_AcceleratedCompositingBlocked DISABLED_AcceleratedCompositingBlocked |
| +#else |
| +#define MAYBE_AcceleratedCompositingBlocked AcceleratedCompositingBlocked |
| +#endif |
| + |
| +IN_PROC_BROWSER_TEST_F(GpuFeatureTest, MAYBE_AcceleratedCompositingBlocked) { |
| + const std::string json_blacklist = |
|
gab
2013/08/27 22:24:12
Moved the test blacklisting directly inside the te
|
| "{\n" |
| " \"name\": \"gpu blacklist\",\n" |
| " \"version\": \"1.0\",\n" |
| @@ -202,19 +206,7 @@ class AcceleratedCompositingBlockedTest : public GpuFeatureTest { |
| " }\n" |
| " ]\n" |
| "}"; |
| - SetupBlacklist(json_blacklist); |
| - } |
| -}; |
| - |
| -#if defined(USE_AURA) |
| -// Compositing is always on for Aura. |
| -#define MAYBE_AcceleratedCompositingBlocked DISABLED_AcceleratedCompositingBlocked |
| -#else |
| -#define MAYBE_AcceleratedCompositingBlocked AcceleratedCompositingBlocked |
| -#endif |
| - |
| -IN_PROC_BROWSER_TEST_F(AcceleratedCompositingBlockedTest, |
| - MAYBE_AcceleratedCompositingBlocked) { |
| + SetupBlacklist(json_blacklist); |
| EXPECT_TRUE(GpuDataManager::GetInstance()->IsFeatureBlacklisted( |
| gpu::GPU_FEATURE_TYPE_ACCELERATED_COMPOSITING)); |
| @@ -364,11 +356,18 @@ IN_PROC_BROWSER_TEST_F(GpuFeatureTest, Canvas2DAllowed) { |
| if (gpu::GPUTestBotConfig::CurrentConfigMatches("XP")) |
| return; |
| - EXPECT_FALSE(GpuDataManager::GetInstance()->IsFeatureBlacklisted( |
| - gpu::GPU_FEATURE_TYPE_ACCELERATED_2D_CANVAS)); |
| + bool is_blacklisted = false; |
| +#if defined(OS_LINUX) && !defined(OS_CHROMEOS) |
| + // Blacklist rule #24 disables accelerated_2d_canvas on Linux. |
| + is_blacklisted = true; |
| +#endif |
| + |
| + EXPECT_EQ(is_blacklisted, |
| + GpuDataManager::GetInstance()->IsFeatureBlacklisted( |
| + gpu::GPU_FEATURE_TYPE_ACCELERATED_2D_CANVAS)); |
| const base::FilePath url(FILE_PATH_LITERAL("feature_canvas2d.html")); |
| - RunEventTest(url, kAcceleratedCanvasCreationEvent, true); |
| + RunEventTest(url, kAcceleratedCanvasCreationEvent, !is_blacklisted); |
| } |
| IN_PROC_BROWSER_TEST_F(GpuFeatureTest, Canvas2DBlocked) { |