Only use pending navigation params for browser-initiated navigations.

content/renderer/render_frame_impl_browsertest.cc

 // Copyright 2015 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 <stdint.h>
 
 #include "base/command_line.h"
 #include "base/debug/leak_annotations.h"
 #include "build/build_config.h"
 #include "content/common/frame_messages.h"
@@ skipping 179 matching lines @@
   SetIsUsingLoFi(frame(), true);
   EXPECT_TRUE(GetMainRenderFrame()->IsUsingLoFi());
   EXPECT_TRUE(frame()->IsUsingLoFi());
 
   blink::WebHistoryItem item;
   item.initialize();
 
   // The main frame's and subframe's LoFi states should stay the same on
   // navigations within the page.
   frame()->didNavigateWithinPage(frame()->GetWebFrame(), item,
-                                 blink::WebStandardCommit);
+                                 blink::WebStandardCommit, true);
   EXPECT_TRUE(frame()->IsUsingLoFi());
   GetMainRenderFrame()->didNavigateWithinPage(
-      GetMainRenderFrame()->GetWebFrame(), item, blink::WebStandardCommit);
+      GetMainRenderFrame()->GetWebFrame(), item, blink::WebStandardCommit,
+      true);
   EXPECT_TRUE(GetMainRenderFrame()->IsUsingLoFi());
 
   // The subframe's LoFi state should not be reset on commit.
   DocumentState* document_state =
       DocumentState::FromDataSource(frame()->GetWebFrame()->dataSource());
   static_cast<NavigationStateImpl*>(document_state->navigation_state())
       ->set_was_within_same_page(false);
 
   frame()->didCommitProvisionalLoad(frame()->GetWebFrame(), item,
                                     blink::WebStandardCommit);
@@ skipping 36 matching lines @@
     EXPECT_EQ(tests[i].type, frame()->getEffectiveConnectionType());
     EXPECT_EQ(tests[i].type,
               GetMainRenderFrame()->getEffectiveConnectionType());
 
     blink::WebHistoryItem item;
     item.initialize();
 
     // The main frame's and subframe's effective connection type should stay the
     // same on navigations within the page.
     frame()->didNavigateWithinPage(frame()->GetWebFrame(), item,
-                                   blink::WebStandardCommit);
+                                   blink::WebStandardCommit, true);
     EXPECT_EQ(tests[i].type, frame()->getEffectiveConnectionType());
     GetMainRenderFrame()->didNavigateWithinPage(
-        GetMainRenderFrame()->GetWebFrame(), item, blink::WebStandardCommit);
+        GetMainRenderFrame()->GetWebFrame(), item, blink::WebStandardCommit,
+        true);
     EXPECT_EQ(tests[i].type, frame()->getEffectiveConnectionType());
 
     // The subframe's effective connection type should not be reset on commit.
     DocumentState* document_state =
         DocumentState::FromDataSource(frame()->GetWebFrame()->dataSource());
     static_cast<NavigationStateImpl*>(document_state->navigation_state())
         ->set_was_within_same_page(false);
 
     frame()->didCommitProvisionalLoad(frame()->GetWebFrame(), item,
                                       blink::WebStandardCommit);
@@ skipping 10 matching lines @@
     EXPECT_EQ(blink::WebEffectiveConnectionType::TypeUnknown,
               GetMainRenderFrame()->getEffectiveConnectionType());
 
     // The subframe would be deleted here after a cross-document navigation.
     // It happens to be left around in this test because this does not simulate
     // the frame detach.
   }
 }
 
 }  // namespace