Move loading methods from WebFrame to WebLocalFrame.

content/shell/renderer/layout_test/blink_test_runner.cc

 // Copyright (c) 2012 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/renderer/layout_test/blink_test_runner.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <clocale>
@@ skipping 1009 matching lines @@
   current_entry_indexes_ = current_entry_indexes;
   CaptureDump();
 }
 
 void BlinkTestRunner::OnReset() {
   LayoutTestRenderThreadObserver::GetInstance()->test_interfaces()->ResetAll();
   Reset(true /* for_new_test */);
   // Navigating to about:blank will make sure that no new loads are initiated
   // by the renderer.
   WebURLRequest request = WebURLRequest(GURL(url::kAboutBlankURL));
-  render_view()->GetWebView()->MainFrame()->LoadRequest(request);
+  if (render_view()->GetWebView()->MainFrame()->IsWebLocalFrame()) {

[dcheng, 2017/06/16 23:54:13]

Hmm... seems racy for OOPIFs then... about:blank commits synchronously here, so
that's fine, but in other renderers...

Maybe this is the best we can do for now.

[Łukasz Anforowicz, 2017/06/19 22:30:42]

I looked closer and it seems that the ShellViewMsg_Reset is only sent to the
current view (well, almost - before this CL the routing id was saved/bound
before sending the IPC;  after this CL this IPC will really always be send to
the current view).

These changes are in
content/shell/browser/layout_test/blink_test_controller.cc
content/shell/browser/layout_test/blink_test_controller.h
content/shell/renderer/layout_test/blink_test_runner.cc
(I've accidentally included a rebase in the most recent patchset...).

+    render_view()->GetWebView()->MainFrame()->ToWebLocalFrame()->LoadRequest(
+        request);
+  }
   Send(new ShellViewHostMsg_ResetDone(routing_id()));
 }
 
 void BlinkTestRunner::OnTestFinishedInSecondaryRenderer() {
   DCHECK(is_main_window_ && render_view()->GetMainRenderFrame());
 
   // Avoid a situation where TestFinished is called twice, because
   // of a racey test finish in 2 secondary renderers.
   test_runner::WebTestInterfaces* interfaces =
       LayoutTestRenderThreadObserver::GetInstance()->test_interfaces();
@@ skipping 23 matching lines @@
 void BlinkTestRunner::ReportLeakDetectionResult(
     const LeakDetectionResult& report) {
   Send(new ShellViewHostMsg_LeakDetectionDone(routing_id(), report));
 }
 
 void BlinkTestRunner::OnDestruct() {
   delete this;
 }
 
 }  // namespace content