Move loading methods from WebFrame to WebLocalFrame.

content/shell/renderer/layout_test/blink_test_runner.cc

Index: content/shell/renderer/layout_test/blink_test_runner.cc
diff --git a/content/shell/renderer/layout_test/blink_test_runner.cc b/content/shell/renderer/layout_test/blink_test_runner.cc
index 91d0cfabf41e7a21ad60a2a91e08073ea9e48030..a798e7071d9f11c4a42e92e834c7d14b7691bb30 100644
--- a/content/shell/renderer/layout_test/blink_test_runner.cc
+++ b/content/shell/renderer/layout_test/blink_test_runner.cc
@@ -1027,7 +1027,10 @@ void BlinkTestRunner::OnReset() {
   // 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()));
 }