Refactor the loading tracking logic in WebContentsImpl.

content/browser/frame_host/frame_tree_node.cc

Index: content/browser/frame_host/frame_tree_node.cc
diff --git a/content/browser/frame_host/frame_tree_node.cc b/content/browser/frame_host/frame_tree_node.cc
index 845aecea62afc5945311f283f029c8d8b6504682..ffeea93ee27862e1673462fa27f50dbd4432a4e9 100644
--- a/content/browser/frame_host/frame_tree_node.cc
+++ b/content/browser/frame_host/frame_tree_node.cc
@@ -34,8 +34,7 @@ FrameTreeNode::FrameTreeNode(FrameTree* frame_tree,
                       manager_delegate),
       frame_tree_node_id_(next_frame_tree_node_id_++),
       parent_(NULL),
-      replication_state_(name),
-      is_loading_(false) {
+      replication_state_(name) {
 }
 
 FrameTreeNode::~FrameTreeNode() {
@@ -115,4 +114,25 @@ bool FrameTreeNode::IsDescendantOf(FrameTreeNode* other) const {
   return false;
 }
 
+bool FrameTreeNode::IsLoading() const {
+  RenderFrameHostImpl* current_frame_host =
+      render_manager_.current_frame_host();
+  RenderFrameHostImpl* pending_frame_host =
+      render_manager_.pending_frame_host();
+
+  DCHECK(current_frame_host != nullptr);
+  if (pending_frame_host != nullptr)

[clamy, 2015/03/02 10:30:58]

Could you add a TODO to have a PlzNavigate specific implementation as this will
not work.

[Fabrice (no longer in Chrome), 2015/03/02 18:01:39]

Done.

+    return current_frame_host->is_loading() || pending_frame_host->is_loading();
+  else
+    return current_frame_host->is_loading();
+}
+
+double FrameTreeNode::GetLoadingProgress() const {
+  RenderFrameHostImpl* current_frame_host =
+      render_manager_.current_frame_host();
+
+  DCHECK(current_frame_host != nullptr);
+  return current_frame_host->loading_progress();
+}
+
 }  // namespace content