If the renderer drops a navigation, make sure the spinner doesn't spin endlessly.

content/browser/frame_host/render_frame_host_delegate.h

Index: content/browser/frame_host/render_frame_host_delegate.h
diff --git a/content/browser/frame_host/render_frame_host_delegate.h b/content/browser/frame_host/render_frame_host_delegate.h
index 6c22d94abf2620ffd350db98afcf79a509a95773..1676f27dce1f0a8f590368ccf79ac8a6520bce17 100644
--- a/content/browser/frame_host/render_frame_host_delegate.h
+++ b/content/browser/frame_host/render_frame_host_delegate.h
@@ -64,6 +64,10 @@ class CONTENT_EXPORT RenderFrameHostDelegate {
   virtual void DidStartLoading(RenderFrameHost* render_frame_host,
                                bool to_different_document) {}
 
+  // The top-level RenderFrame stopped loading a page. This corresponds to

[nasko, 2014/12/09 18:13:33]

Is this going to be only for top-level frames? I don't see this enforced in
code. I also imagine we will have frames report start/stop loading events, since
browser side will know the full picture, so I doubt longer term this will be
isolated to top-level only.

[Avi (use Gerrit), 2014/12/09 20:45:27]

No, but the comment was stolen from DidStartLoading, and _that_ is called from
RenderFrameHostImpl::Navigate for all frames _also_. Will fix the comments.

+  // Blink's notion of the throbber stopping.
+  virtual void DidStopLoading(RenderFrameHost* render_frame_host) {}
+
   // The RenderFrameHost has been swapped out.
   virtual void SwappedOut(RenderFrameHost* render_frame_host) {}