Split DumpPixelsAsync in pixel_dump.h into more granular functions.

content/shell/test_runner/test_runner_for_specific_view.cc

 // Copyright 2016 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 "content/shell/test_runner/test_runner_for_specific_view.h"
 
 #include <stddef.h>
 #include <limits>
 #include <utility>
 
@@ skipping 215 matching lines @@
   test_runner::LayoutAndPaintAsyncThen(
       web_view()->MainFrame()->ToWebLocalFrame()->FrameWidget(),
       CreateClosureThatPostsV8Callback(callback));
 }
 
 void TestRunnerForSpecificView::CapturePixelsAsyncThen(
     v8::Local<v8::Function> callback) {
   v8::UniquePersistent<v8::Function> persistent_callback(
       blink::MainThreadIsolate(), callback);
 
+  CHECK(web_view()->MainFrame()->IsWebLocalFrame())
+      << "Layout tests harness doesn't currently support running "
+      << "testRuner.capturePixelsAsyncThen from an OOPIF";
+
   web_view_test_proxy_base_->test_interfaces()
       ->GetTestRunner()
       ->DumpPixelsAsync(
-          web_view(),
+          web_view()->MainFrame()->ToWebLocalFrame(),
           base::Bind(&TestRunnerForSpecificView::CapturePixelsCallback,
                      weak_factory_.GetWeakPtr(),
                      base::Passed(std::move(persistent_callback))));
 }
 
 void TestRunnerForSpecificView::CapturePixelsCallback(
     v8::UniquePersistent<v8::Function> callback,
     const SkBitmap& snapshot) {
   v8::Isolate* isolate = blink::MainThreadIsolate();
   v8::HandleScope handle_scope(isolate);
@@ skipping 32 matching lines @@
   PostV8CallbackWithArgs(std::move(callback), arraysize(argv), argv);
 }
 
 void TestRunnerForSpecificView::CopyImageAtAndCapturePixelsAsyncThen(
     int x,
     int y,
     v8::Local<v8::Function> callback) {
   v8::UniquePersistent<v8::Function> persistent_callback(
       blink::MainThreadIsolate(), callback);
 
+  // TODO(lukasza): Support image capture in OOPIFs for
+  // https://crbug.com/477150.
+  CHECK(web_view()->MainFrame()->IsWebLocalFrame())
+      << "Layout tests harness doesn't support calling "
+      << "testRunner.copyImageAtAndCapturePixelsAsyncThen from an OOPIF.";
+
   CopyImageAtAndCapturePixels(
-      web_view(), x, y,
+      web_view()->MainFrame()->ToWebLocalFrame(), x, y,
       base::Bind(&TestRunnerForSpecificView::CapturePixelsCallback,
                  weak_factory_.GetWeakPtr(),
                  base::Passed(std::move(persistent_callback))));
 }
 
 void TestRunnerForSpecificView::GetManifestThen(
     v8::Local<v8::Function> callback) {
   if (!web_view()->MainFrame()->IsWebLocalFrame()) {
     CHECK(false) << "This function cannot be called if the main frame is not a "
                     "local frame.";
@@ skipping 389 matching lines @@
 
 blink::WebView* TestRunnerForSpecificView::web_view() {
   return web_view_test_proxy_base_->web_view();
 }
 
 WebTestDelegate* TestRunnerForSpecificView::delegate() {
   return web_view_test_proxy_base_->delegate();
 }
 
 }  // namespace test_runner