Reset the loading frame count (etc) when starting a new main frame navigation.

content/browser/web_contents/web_contents_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/debug/crash_logging.h"
@@ skipping 2718 matching lines @@
       static_cast<RenderFrameHostImpl*>(render_frame_message_source_);
   FOR_EACH_OBSERVER(
       WebContentsObserver, observers_, DidFinishLoad(rfh, validated_url));
 }
 
 void WebContentsImpl::OnDidStartLoading(bool to_different_document) {
   RenderFrameHostImpl* rfh =
       static_cast<RenderFrameHostImpl*>(render_frame_message_source_);
   int64 render_frame_id = rfh->frame_tree_node()->frame_tree_node_id();
 
+  // Any main frame load to a new document should reset the load progress, since
+  // it will replace the current page and any frames.
+  if (to_different_document && !rfh->GetParent()) {
+    ResetLoadProgressState();
+    loading_frames_in_progress_ = 0;

[Charlie Reis, 2014/12/19 19:47:48]

Note: I didn't put this line inside ResetLoadProgressState because that would
have affected OnDidStopLoading's behavior.

[Avi (use Gerrit), 2014/12/19 19:54:54]

Acknowledged.

+  }
+
   // It is possible to get multiple calls to OnDidStartLoading that don't have
   // corresponding calls to OnDidStopLoading:
   // - With "swappedout://" URLs, this happens when a RenderView gets swapped
   //   out for a cross-process navigation, and it turns into a placeholder for
   //   one being rendered in a different process.
   // - Also, there might be more than one RenderFrameHost sharing the same
   //   FrameTreeNode (and thus sharing its ID) each sending a start.
   // - But in the future, once clamy@ moves navigation network requests to the
   //   browser process, there's a good chance that callbacks about starting and
   //   stopping will all be handled by the browser. When that happens, there
@@ skipping 1638 matching lines @@
   node->render_manager()->ResumeResponseDeferredAtStart();
 }
 
 void WebContentsImpl::SetForceDisableOverscrollContent(bool force_disable) {
   force_disable_overscroll_content_ = force_disable;
   if (view_)
     view_->SetOverscrollControllerEnabled(CanOverscrollContent());
 }
 
 }  // namespace content