Support error handling for Surface readbacks

chrome/browser/ui/fullscreen/flash_fullscreen_interactive_browsertest.cc

 // Copyright 2014 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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "base/time/time.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 219 matching lines @@
             &FlashFullscreenInteractiveBrowserTest::CheckBitmapForFillColor,
             expected_color,
             &is_expected_color,
             base::MessageLoop::QuitClosure()),
         kN32_SkColorType);
     content::RunMessageLoop();
 
     return is_expected_color;
   }
 
-  static void CheckBitmapForFillColor(SkColor expected_color,
-                                      bool* is_expected_color,
-                                      const base::Closure& done_cb,
-                                      bool success,
-                                      const SkBitmap& bitmap) {
-    if (success) {
+  static void CheckBitmapForFillColor(
+      SkColor expected_color,
+      bool* is_expected_color,
+      const base::Closure& done_cb,
+      const SkBitmap& bitmap,
+      const content::ReadbackResponse& response) {
+    if (response == content::READBACK_SUCCESS) {
       SkAutoLockPixels lock_pixels(bitmap);
       if (bitmap.width() > 0 && bitmap.height() > 0)
         *is_expected_color = (bitmap.getColor(0, 0) == expected_color);
     }
     done_cb.Run();
   }
 
   DISALLOW_COPY_AND_ASSIGN(FlashFullscreenInteractiveBrowserTest);
 };
 
@@ skipping 83 matching lines @@
   ASSERT_TRUE(ObserveFlashFillColor(SK_ColorRED));
   PressSpacebar();
   ASSERT_TRUE(ObserveFlashFillColor(SK_ColorBLUE));
 
   // Test that the Escape key is handled as an exit fullscreen command while the
   // Flash widget has the focus.
   EXPECT_TRUE(ObserveFlashHasFocus(first_tab_contents, true));
   PressEscape();
   EXPECT_TRUE(ObserveTabIsInFullscreen(false));
 }