Fix NavigationHandle being thrown out and recreated (without all the state) after rapid navigations.

chrome/browser/ssl/ssl_browser_tests.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 <utility>
 
 #include "base/base64.h"
 #include "base/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 4334 matching lines @@
   CheckAuthenticatedState(tab, AuthState::NONE);
 
   content::TestNavigationObserver observer(tab);
   ASSERT_TRUE(content::ExecuteScript(
       tab, "location.replace(window.location.href + '#1')"));
   observer.Wait();
   content::WaitForLoadStop(tab);
   CheckAuthenticatedState(tab, AuthState::NONE);
 }
 
+// Checks that a restore followed immediately by a history navigation doesn't
+// lose SSL state.
+IN_PROC_BROWSER_TEST_F(SSLUITest, RestoreThenNavigateHasSSLState) {
+  // This race condition only happens with PlzNavigate.
+  if (!content::IsBrowserSideNavigationEnabled())
+    return;
+  ASSERT_TRUE(https_server_.Start());
+  GURL url1(https_server_.GetURL("/ssl/google.html"));
+  GURL url2(https_server_.GetURL("/ssl/page_with_refs.html"));
+  ui_test_utils::NavigateToURLWithDisposition(
+      browser(), url1, WindowOpenDisposition::NEW_FOREGROUND_TAB,
+      ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
+  ui_test_utils::NavigateToURL(browser(), url2);
+  chrome::CloseTab(browser());
+
+  content::WindowedNotificationObserver tab_added_observer(
+      chrome::NOTIFICATION_TAB_PARENTED,
+      content::NotificationService::AllSources());
+  chrome::RestoreTab(browser());
+  tab_added_observer.Wait();
+
+  WebContents* tab = browser()->tab_strip_model()->GetActiveWebContents();
+  content::TestNavigationManager observer(tab, url1);
+  chrome::GoBack(browser(), WindowOpenDisposition::CURRENT_TAB);
+  observer.WaitForNavigationFinished();
+  CheckAuthenticatedState(tab, AuthState::NONE);
+}
+
 // Simulate the URL changing when the user presses enter in the omnibox. This
 // could happen when the user's login is expired and the server redirects them
 // to a login page. This will be considered a SAME_PAGE navigation but we do
 // want to update the SSL state.
 IN_PROC_BROWSER_TEST_F(SSLUITest, SamePageHasSSLState) {
   ASSERT_TRUE(embedded_test_server()->Start());
   ASSERT_TRUE(https_server_.Start());
 
   WebContents* tab = browser()->tab_strip_model()->GetActiveWebContents();
 
@@ skipping 749 matching lines @@
 
 // Visit a page over https that contains a frame with a redirect.
 
 // XMLHttpRequest insecure content in synchronous mode.
 
 // XMLHttpRequest insecure content in asynchronous mode.
 
 // XMLHttpRequest over bad ssl in synchronous mode.
 
 // XMLHttpRequest over OK ssl in synchronous mode.