Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(202)

Unified Diff: content/browser/frame_host/navigation_controller_impl.cc

Issue 2974553002: Fix NavigationHandle being thrown out and recreated (without all the state) after rapid navigations. (Closed)
Patch Set: fix Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/frame_host/navigation_controller_impl.cc
diff --git a/content/browser/frame_host/navigation_controller_impl.cc b/content/browser/frame_host/navigation_controller_impl.cc
index f15af73e7d65241e8125e0f1f27af6dd9a1ed046..1ec2414cf68561c417824d2b29ea87b0cbba17bc 100644
--- a/content/browser/frame_host/navigation_controller_impl.cc
+++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -1297,10 +1297,7 @@ void NavigationControllerImpl::RendererDidNavigateToExistingPage(
} else {
// When restoring a tab, the serialized NavigationEntry doesn't have the
// SSL state.
- // Only copy in the restore case since this code path can be taken during
- // navigation. See http://crbug.com/727892
- if (was_restored)
- entry->GetSSL() = handle->ssl_status();
+ entry->GetSSL() = handle->ssl_status();
}
} else {
// This is renderer-initiated. The only kinds of renderer-initated

Powered by Google App Engine
This is Rietveld 408576698