Move DidCommitProvisionalLoad code from RenderView to RenderFrame.

content/browser/frame_host/navigator_delegate.h

 // 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.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_
 
+#include "base/strings/string16.h"
 #include "content/public/browser/invalidate_type.h"
 #include "content/public/browser/navigation_controller.h"
+#include "content/public/common/page_transition_types.h"
+
+class GURL;
+struct FrameHostMsg_DidFailProvisionalLoadWithError_Params;
+struct FrameHostMsg_DidCommitProvisionalLoad_Params;

[Charlie Reis, 2014/02/05 23:30:37]

nit: Alphabetize

[nasko, 2014/02/06 01:55:13]

Done.

 
 namespace content {
 
-class RenderFrameHost;
+class RenderFrameHostImpl;
+struct LoadCommittedDetails;
 
 // A delegate API used by Navigator to notify its embedder of navigation
 // related events.
-class NavigatorDelegate {
+class CONTENT_EXPORT NavigatorDelegate {
  public:
   // The RenderFrameHost started a provisional load for the frame
   // represented by |render_frame_host|.
   virtual void DidStartProvisionalLoad(
       RenderFrameHostImpl* render_frame_host,
       int64 frame_id,
       int64 parent_frame_id,
       bool is_main_frame,
       const GURL& validated_url,
       bool is_error_page,
       bool is_iframe_srcdoc) {}
 
   // A provisional load in |render_frame_host| failed.
   virtual void DidFailProvisionalLoadWithError(
       RenderFrameHostImpl* render_frame_host,
       const FrameHostMsg_DidFailProvisionalLoadWithError_Params& params) {}
 
   // A redirect was processed in |render_frame_host| during a provisional load.
   virtual void DidRedirectProvisionalLoad(
       RenderFrameHostImpl* render_frame_host,
       const GURL& validated_target_url) {}
 
+  // A commit was processed in |render_frame_host|.

[Charlie Reis, 2014/02/05 23:30:37]

A navigation was committed in...

[nasko, 2014/02/06 01:55:13]

Done.

+  virtual void DidCommitProvisionalLoad(
+      int64 frame_id,
+      const base::string16& frame_unique_name,
+      bool is_main_frame,
+      const GURL& url,
+      PageTransition transition_type,
+      RenderFrameHostImpl* render_frame_host) {}
+
+  // Handles post-navigation tasks in navigation AFTER the entry has been
+  // committed to the navigation controller. Note that the navigation entry is

[Charlie Reis, 2014/02/05 23:30:37]

nit: NavigationController, NavigationEntry

[nasko, 2014/02/06 01:55:13]

Done.

+  // not provided since it may be invalid/changed after being committed. The
+  // current navigation entry is in the NavigationController at this point.

[Charlie Reis, 2014/02/05 23:30:37]

What do you mean by "current" and "in the NavigationController?"  Maybe we
should say that the NavigationController's last committed entry is for this
navigation?

[nasko, 2014/02/06 01:55:13]

This is a comment that was just moved from WebContentsImpl.h, but I agree that
last committed entry makes more sense.

+  virtual void DidNavigateMainFramePostCommit(
+      const LoadCommittedDetails& details,
+      const FrameHostMsg_DidCommitProvisionalLoad_Params& params) {}
+  virtual void DidNavigateAnyFramePostCommit(
+      RenderFrameHostImpl* render_frame_host,
+      const LoadCommittedDetails& details,
+      const FrameHostMsg_DidCommitProvisionalLoad_Params& params) {}
+
+  virtual void SetMainFrameMimeType(const std::string& mime_type) {}
+  virtual bool CanOverscrollContent();
+
   // Notification to the Navigator embedder that navigation state has
   // changed. This method corresponds to
   // WebContents::NotifyNavigationStateChanged.
   virtual void NotifyChangedNavigationState(InvalidateTypes changed_flags) {}
 
   // Notifies the Navigator embedder that it is beginning to navigate a frame.
   virtual void AboutToNavigateRenderFrame(
       RenderFrameHostImpl* render_frame_host) {}
 
   // Notifies the Navigator embedder that a navigation to pending
   // NavigationEntry has started in the browser process.
   virtual void DidStartNavigationToPendingEntry(
       RenderFrameHostImpl* render_frame_host,
       const GURL& url,
       NavigationController::ReloadType reload_type) {}
 };
 
 }  // namspace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_DELEGATE_H_