Reenable CaptureScreenshot test for mac/msan builds.

content/browser/devtools/protocol/devtools_protocol_browsertest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/base64.h"
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
@@ skipping 359 matching lines @@
 class CaptureScreenshotTest : public DevToolsProtocolTest {
  private:
 #if !defined(OS_ANDROID)
   void SetUpCommandLine(base::CommandLine* command_line) override {
     command_line->AppendSwitch(switches::kEnablePixelOutputInTests);
   }
 #endif
 };
 
 // Does not link on Android
-#if defined(OS_ANDROID)
-#define MAYBE_CaptureScreenshot DISABLED_CaptureScreenshot
-#elif defined(OS_MACOSX)  // Fails on 10.9. http://crbug.com/430620
-#define MAYBE_CaptureScreenshot DISABLED_CaptureScreenshot
-#elif defined(MEMORY_SANITIZER)
-// Also fails under MSAN. http://crbug.com/423583
-#define MAYBE_CaptureScreenshot DISABLED_CaptureScreenshot
-#else
-#define MAYBE_CaptureScreenshot CaptureScreenshot
-#endif
-IN_PROC_BROWSER_TEST_F(CaptureScreenshotTest, MAYBE_CaptureScreenshot) {
+#if !defined(OS_ANDROID)
+IN_PROC_BROWSER_TEST_F(CaptureScreenshotTest, CaptureScreenshot) {
   shell()->LoadURL(GURL("about:blank"));
   Attach();
   EXPECT_TRUE(content::ExecuteScript(
       shell()->web_contents()->GetRenderViewHost(),
       "document.body.style.background = '#123456'"));
   SendCommand("Page.captureScreenshot", nullptr);
   std::string base64;
   EXPECT_TRUE(result_->GetString("data", &base64));
   std::string png;
   EXPECT_TRUE(base::Base64Decode(base64, &png));
   SkBitmap bitmap;
   gfx::PNGCodec::Decode(reinterpret_cast<const unsigned char*>(png.data()),
                         png.size(), &bitmap);
   SkColor color(bitmap.getColor(0, 0));
-  EXPECT_TRUE(std::abs(0x12-(int)SkColorGetR(color)) <= 1);
-  EXPECT_TRUE(std::abs(0x34-(int)SkColorGetG(color)) <= 1);
-  EXPECT_TRUE(std::abs(0x56-(int)SkColorGetB(color)) <= 1);
+  EXPECT_GE(1, std::abs(0x12-(int)SkColorGetR(color)));
+  EXPECT_GE(1, std::abs(0x34-(int)SkColorGetG(color)));
+  EXPECT_GE(1, std::abs(0x56-(int)SkColorGetB(color)));
+  color = bitmap.getColor(1, 1);
+  EXPECT_GE(1, std::abs(0x12-(int)SkColorGetR(color)));
+  EXPECT_GE(1, std::abs(0x34-(int)SkColorGetG(color)));
+  EXPECT_GE(1, std::abs(0x56-(int)SkColorGetB(color)));
 }
+#endif
 
 #if defined(OS_ANDROID)
 // Disabled, see http://crbug.com/469947.
 IN_PROC_BROWSER_TEST_F(DevToolsProtocolTest, DISABLED_SynthesizePinchGesture) {
   GURL test_url = GetTestUrl("devtools", "synthetic_gesture_tests.html");
   NavigateToURLBlockUntilNavigationsComplete(shell(), test_url, 1);
   Attach();
 
   int old_width;
   ASSERT_TRUE(content::ExecuteScriptAndExtractInt(
@@ skipping 462 matching lines @@
   params->SetString("policy", "advance");
   params->SetInteger("budget", 1000);
   SendCommand("Emulation.setVirtualTimePolicy", std::move(params), true);
 
   WaitForNotification("Emulation.virtualTimeBudgetExpired");
 
   EXPECT_THAT(console_messages_, ElementsAre("before", "done", "after"));
 }
 
 }  // namespace content