PlzNavigate: remove extraneous DidStartLoading IPC

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 409a2a84932d6b0330cadc30c3eaa7e9121a972a..a38506ced1347355ef22c2822d59e6a3c6a6c22b 100644
--- a/content/browser/frame_host/frame_tree_node.cc
+++ b/content/browser/frame_host/frame_tree_node.cc
@@ -235,6 +235,13 @@ bool FrameTreeNode::IsLoading() const {
           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;