PlzNavigate: remove extraneous DidStartLoading IPC

content/browser/frame_host/frame_tree_node.cc

 // Copyright 2013 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/frame_host/frame_tree_node.h"
 
 #include <queue>
 
 #include "base/command_line.h"
 #include "base/profiler/scoped_tracker.h"
@@ skipping 217 matching lines @@
       render_manager_.current_frame_host();
   RenderFrameHostImpl* pending_frame_host =
       render_manager_.pending_frame_host();
 
   DCHECK(current_frame_host);
 
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kEnableBrowserSideNavigation)) {
     if (navigation_request_)
       return true;
+
+    // Check if a navigation is about ot commit in the speculative

[Fabrice (no longer in Chrome), 2015/07/07 15:16:35]

nit: s/ot/to/

[nasko, 2015/07/07 15:18:57]

nit: Does it really matter whether it is about to commit or not? Technically, if
the speculative RFH says it is loading, we should take it into account, right?

[clamy, 2015/07/07 15:23:58]

Well it should only tell us that it is loading when it is about to commit. Since
it is not adding that much, I just removed the comment.

+    // RenderFrameHost.
+    RenderFrameHostImpl* speculative_frame_host =
+        render_manager_.speculative_frame_host();
+    if (speculative_frame_host && speculative_frame_host->is_loading())
+      return true;
   } else {
     if (pending_frame_host && pending_frame_host->is_loading())
       return true;
   }
   return current_frame_host->is_loading();
 }
 
 bool FrameTreeNode::CommitPendingSandboxFlags() {
   bool did_change_flags =
       effective_sandbox_flags_ != replication_state_.sandbox_flags;
@@ skipping 101 matching lines @@
       FROM_HERE_WITH_EXPLICIT_FUNCTION(
           "465796 FrameTreeNode::DidStopLoading::End"));
 }
 
 void FrameTreeNode::DidChangeLoadProgress(double load_progress) {
   loading_progress_ = load_progress;
   frame_tree_->UpdateLoadProgress();
 }
 
 }  // namespace content