Support error handling for Surface readbacks

chrome/browser/ui/pdf/pdf_browsertest_base.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 "chrome/browser/ui/pdf/pdf_browsertest_base.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/path_service.h"
@@ skipping 93 matching lines @@
 
   content::RunMessageLoop();
 
   if (snapshot_different_) {
     LOG(INFO) << "Rendering didn't match, see result "
               << snapshot_filename_.value();
   }
   return !snapshot_different_;
 }
 
-void PDFBrowserTest::CopyFromBackingStoreCallback(bool success,
-                                                  const SkBitmap& bitmap) {
+void PDFBrowserTest::CopyFromBackingStoreCallback(
+    const SkBitmap& bitmap,
+    content::ReadbackResponse response) {
   base::MessageLoopForUI::current()->Quit();
-  ASSERT_EQ(success, true);
+  ASSERT_EQ(response, content::READBACK_SUCCESS);
   base::FilePath reference = ui_test_utils::GetTestFilePath(
       base::FilePath(FILE_PATH_LITERAL("pdf_private")),
       base::FilePath().AppendASCII(expected_filename_));
   base::File::Info info;
   ASSERT_TRUE(base::GetFileInfo(reference, &info));
   int size = static_cast<size_t>(info.size);
   scoped_ptr<char[]> data(new char[size]);
   ASSERT_EQ(size, base::ReadFile(reference, data.get(), size));
 
   int w, h;
@@ skipping 60 matching lines @@
   // Calling RenderWidgetHost::CopyFromBackingStore() with the GPU enabled
   // fails on Linux.
   command_line->AppendSwitch(switches::kDisableGpu);
 #endif
 
 #if defined(OS_CHROMEOS)
   // Also need on CrOS in addition to disabling the GPU above.
   command_line->AppendSwitch(switches::kUIDisableThreadedCompositing);
 #endif
 }