Move DidFailProvisionalLoad handling from RenderView(Host) to RenderFrame(Host).

content/browser/frame_host/navigator_impl.cc

 // Copyright 2013 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/browser/frame_host/navigator_impl.h"
 
 #include "base/command_line.h"
+#include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/frame_host/frame_tree_node.h"
 #include "content/browser/frame_host/navigation_controller_impl.h"
 #include "content/browser/frame_host/navigation_entry_impl.h"
 #include "content/browser/frame_host/navigator_delegate.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/site_instance_impl.h"
+#include "content/common/frame_messages.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/invalidate_type.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/url_constants.h"
 
 namespace content {
 
 NavigatorImpl::NavigatorImpl(
@@ skipping 59 matching lines @@
   }
 
   if (delegate_) {
     // Notify the observer about the start of the provisional load.
     delegate_->DidStartProvisionalLoad(
         render_frame_host, frame_id, parent_frame_id, is_main_frame,
         validated_url, is_error_page, is_iframe_srcdoc);
   }
 }
 
+
+void NavigatorImpl::DidFailProvisionalLoadWithError(
+    RenderFrameHostImpl* render_frame_host,
+    const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params) {
+  VLOG(1) << "Failed Provisional Load: " << params.url.possibly_invalid_spec()
+          << ", error_code: " << params.error_code
+          << ", error_description: " << params.error_description
+          << ", is_main_frame: " << params.is_main_frame
+          << ", showing_repost_interstitial: " <<
+            params.showing_repost_interstitial
+          << ", frame_id: " << params.frame_id;
+  GURL validated_url(params.url);
+  RenderProcessHost* render_process_host = render_frame_host->GetProcess();
+  render_process_host->FilterURL(false, &validated_url);
+
+  if (net::ERR_ABORTED == params.error_code) {
+    // EVIL HACK ALERT! Ignore failed loads when we're showing interstitials.
+    // This means that the interstitial won't be torn down properly, which is
+    // bad. But if we have an interstitial, go back to another tab type, and
+    // then load the same interstitial again, we could end up getting the first
+    // interstitial's "failed" message (as a result of the cancel) when we're on
+    // the second one.
+    //
+    // We can't tell this apart, so we think we're tearing down the current page

[Charlie Reis, 2014/01/06 19:07:17]

Please keep the first sentence (and put it at the end of the previous
paragraph).

nit: later one -> later on.

[nasko, 2014/01/06 19:16:31]

Done.

+    // which will cause a crash later one. There is also some code in
+    // RenderFrameHostManager::RendererAbortedProvisionalLoad that is commented

[Charlie Reis, 2014/01/06 18:13:35]

We can remove this mention of RFHM::RendererAbortedProvisionalLoad now that it's
gone.

[nasko, 2014/01/06 18:47:03]

Done.

+    // out because of this problem.
+    //
+    // http://code.google.com/p/chromium/issues/detail?id=2855
+    // Because this will not tear down the interstitial properly, if "back" is
+    // back to another tab type, the interstitial will still be somewhat alive
+    // in the previous tab type. If you navigate somewhere that activates the
+    // tab with the interstitial again, you'll see a flash before the new load
+    // commits of the interstitial page.
+    RenderFrameHostManager* root_rfhm = render_frame_host->frame_tree_node()

[Charlie Reis, 2014/01/06 18:13:35]

Nit: I wonder if there's a better way to format this line without all the
newline+arrows, though it's not a big deal.  Maybe we could grab the root and
then check root->render_manager()->interstitial_page() != NULL?

(Side note: I'd love to find a way to remove this whole hack, but that's better
to do after we move it.)

[nasko, 2014/01/06 18:47:03]

Done.

+                                            ->frame_tree()
+                                            ->root()
+                                            ->render_manager();
+    if (root_rfhm->interstitial_page() != NULL) {
+      LOG(WARNING) << "Discarding message during interstitial.";
+      return;
+    }

[Charlie Reis, 2014/01/06 18:13:35]

Let's put the useful part of the RFHM::RendererAbortedProvisionalLoad comment
here.

[nasko, 2014/01/06 18:47:03]

Done.

+  }
+
+  // Do not usually clear the pending entry if one exists, so that the user's
+  // typed URL is not lost when a navigation fails or is aborted.  However, in
+  // cases that we don't show the pending entry (e.g., renderer-initiated
+  // navigations in an existing tab), we don't keep it around.  That prevents
+  // spoofs on in-page navigations that don't go through
+  // DidStartProvisionalLoadForFrame.
+  // In general, we allow the view to clear the pending entry and typed URL if
+  // the user requests (e.g., hitting Escape with focus in the address bar).
+  // Note: don't touch the transient entry, since an interstitial may exist.
+  if (controller_->GetPendingEntry() != controller_->GetVisibleEntry())
+    controller_->DiscardPendingEntry();
+
+  delegate_->DidFailProvisionalLoadWithError(render_frame_host, params);
+}
+
 }  // namespace content