Reland of Refactor the loading tracking logic in WebContentsImpl.

content/browser/frame_host/frame_tree_node.h

Index: content/browser/frame_host/frame_tree_node.h
diff --git a/content/browser/frame_host/frame_tree_node.h b/content/browser/frame_host/frame_tree_node.h
index 370e4e3819f4362981aa58d33b2f76b34526152a..2c0c9d672e71eaa9833545bccf1cffbedb04a7ee 100644
--- a/content/browser/frame_host/frame_tree_node.h
+++ b/content/browser/frame_host/frame_tree_node.h
@@ -30,7 +30,6 @@ class RenderFrameHostImpl;
 // are frame-specific (as opposed to page-specific).
 class CONTENT_EXPORT FrameTreeNode {
  public:
-
   FrameTreeNode(FrameTree* frame_tree,
                 Navigator* navigator,
                 RenderFrameHostDelegate* render_frame_delegate,
@@ -113,19 +112,18 @@ class CONTENT_EXPORT FrameTreeNode {
     return replication_state_;
   }
 
-  void set_is_loading(bool is_loading) {
-    is_loading_ = is_loading;
-  }
-  bool is_loading() const {
-    return is_loading_;
-  }
-
   RenderFrameHostImpl* current_frame_host() const {
     return render_manager_.current_frame_host();
   }
 
   bool IsDescendantOf(FrameTreeNode* other) const;
 
+  // Returns true if this frame is in a loading state.
+  bool IsLoading() const;
+
+  // Returns the loading progress of this frame.
+  double GetLoadingProgress() const;
+
  private:
   void set_parent(FrameTreeNode* parent) { parent_ = parent; }
 
@@ -141,7 +139,7 @@ class CONTENT_EXPORT FrameTreeNode {
 
   // Manages creation and swapping of RenderFrameHosts for this frame.  This
   // must be declared before |children_| so that it gets deleted after them.
-  //  That's currently necessary so that RenderFrameHostImpl's destructor can
+  // That's currently necessary so that RenderFrameHostImpl's destructor can
   // call GetProcess.
   RenderFrameHostManager render_manager_;
 
@@ -176,15 +174,6 @@ class CONTENT_EXPORT FrameTreeNode {
   // flags when a navigation for this frame commits.
   SandboxFlags effective_sandbox_flags_;
 
-  // Boolean value indicating whether the frame is in the process of loading
-  // a document or not. In cross-process transfer navigation the DidStartLoading
-  // message is received from both existing RenderFrame and from the pending
-  // RenderFrame. However, there will be only one DidStopLoading message sent by
-  // the pending-which-becomes-current RenderFrame. Since both renderers belong
-  // to the FrameTreeNode, it is better to ask it about the loading status than
-  // RenderFrameHost or using a counter to balance the events out.
-  bool is_loading_;
-
   DISALLOW_COPY_AND_ASSIGN(FrameTreeNode);
 };