Remove the is_loading_ field from 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 df18cf15ee93d96b4e2e3223710f36c974448c19..81dd2cf90e17519ce5cd1f7eefa30d6cc086bb54 100644
--- a/content/browser/frame_host/frame_tree_node.h
+++ b/content/browser/frame_host/frame_tree_node.h
@@ -236,6 +236,15 @@ class CONTENT_EXPORT FrameTreeNode {
   // time and notifies observers.
   void DidFocus();
 
+  // Called when a RenderViewHost was terminated. This will inform the current
+  // or the pending RenderFrameHost of the RenderViewHost termination should
+  // they be associated with |render_view_host|.
+  void RenderViewTerminated(RenderViewHostImpl* render_view_host);

[nasko, 2016/01/15 01:19:16]

Why do we need this method?

[clamy, 2016/01/15 16:47:44]

Method removed. The loading state is now reset in RFHM::RenderProcessGone.

+
+  // Called when the user closed the modal dialogue for BeforeUnload. This

[nasko, 2016/01/15 01:19:16]

Let's add to the comment that it is called only when the user cancelled the
navigation.

[clamy, 2016/01/15 16:47:44]

Done.

+  // should stop the loads.
+  void BeforeUnloadCanceled();
+
  private:
   class OpenerDestroyedObserver;