Move DidStartLoading, DidStopLoading, DidChangeLoadProgress to RFHI.

content/browser/frame_host/navigator_delegate.h

Index: content/browser/frame_host/navigator_delegate.h
diff --git a/content/browser/frame_host/navigator_delegate.h b/content/browser/frame_host/navigator_delegate.h
index 48501f487ff3c5293dc86ef0a4a7c9465d01d517..c9802032eacb6a7c33f25305d12e2dbfc1558446 100644
--- a/content/browser/frame_host/navigator_delegate.h
+++ b/content/browser/frame_host/navigator_delegate.h
@@ -17,6 +17,7 @@ struct FrameHostMsg_DidFailProvisionalLoadWithError_Params;
 
 namespace content {
 
+class FrameTreeNode;
 class RenderFrameHostImpl;
 struct LoadCommittedDetails;
 struct OpenURLParams;
@@ -99,6 +100,20 @@ class CONTENT_EXPORT NavigatorDelegate {
   // Returns whether URLs for aborted browser-initiated navigations should be
   // preserved in the omnibox.  Defaults to false.
   virtual bool ShouldPreserveAbortedURLs();
+
+  // A RenderFrame in the specified |frame_tree_node| started loading a new

[Charlie Reis, 2015/04/15 23:37:53]

nit: RenderFrameHost.

[Fabrice (no longer in Chrome), 2015/04/16 13:55:25]

Done.

+  // document. This correponds to Blink's notion of the throbber starting.
+  // |to_different_document| will be true unless the load is a fragment
+  // navigation, or triggered by history.pushState/replaceState.
+  virtual void DidStartLoading(FrameTreeNode* frame_tree_node,
+                               bool to_different_document) {}
+
+  // A document stopped loading. This corresponds to Blink's notion of the
+  // throbber stopping.
+  virtual void DidStopLoading() {}

[nasko, 2015/04/15 19:17:05]

Should this method take FTN as a parameter to be consistent with
DidStartLoading? 

[Charlie Reis, 2015/04/15 23:37:53]

I think it's probably ok to omit it until it's needed.

[Fabrice (no longer in Chrome), 2015/04/16 13:55:25]

I think the long term goal is to remove the FrameTreeNode from DidStartLoading
actually, if we remove the accessibility manager call. Semantically, we
shouldn't need to know about the FrameTreeNode at this point.

[nasko, 2015/04/16 16:23:48]

Acknowledged.

+
+  // The load progress was changed.
+  virtual void DidChangeLoadProgress() {}
 };
 
 }  // namspace content