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

content/browser/frame_host/render_frame_host_impl.h

Index: content/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index ebe50a6878761c1daccf3b3c125bb433d053e38b..6e33f5d38865560d48ed30747ba5a1290983acad 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -12,6 +12,7 @@
 #include "content/public/browser/render_frame_host.h"
 
 class GURL;
+struct FrameHostMsg_DidFailProvisionalLoadWithError_Params;
 
 namespace base {
 class FilePath;
@@ -54,6 +55,7 @@ class CONTENT_EXPORT RenderFrameHostImpl : public RenderFrameHost {
 
   RenderViewHostImpl* render_view_host() { return render_view_host_; }
   RenderFrameHostDelegate* delegate() { return delegate_; }
+  FrameTreeNode* frame_tree_node() { return frame_tree_node_; }

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

Do we need to expose FrameTreeNode yet?  Looks like we're just using it to get
to FrameTree, which we could expose instead.

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

Exposing FrameTreeNode seems more logical, since it is the object closest to RFH
and allows us to get to FrameTree if needed. Conceptually, RFH knows which
FrameTreeNode it belongs to, so it makes sense to expose it. I also believe we
will need it for other cases in the near future.

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

That's fair.  Let's leave it as is.

 
   // This function is called when this is a swapped out RenderFrameHost that
   // lives in the same process as the parent frame. The
@@ -89,6 +91,8 @@ class CONTENT_EXPORT RenderFrameHostImpl : public RenderFrameHost {
                                          int64 parent_frame_id,
                                          bool main_frame,
                                          const GURL& url);
+  void OnDidFailProvisionalLoadWithError(
+      const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params);
 
   bool is_swapped_out() { return is_swapped_out_; }