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

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 1048963002: Fix incorrect creation of duplicate navigation entries for repeated page load failures. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@committype3
Patch Set: works Created 5 years, 9 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/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 6626ab0c93e3221f62e0920b824deca043d82f7c..a50dd0f90b4d6c033f74525bf2282ec700c7c748 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -3933,7 +3933,7 @@ void WebContentsImpl::DidAccessInitialDocument() {
// to let the user edit it and try again. Clear it now that content might
// show up underneath it.
if (!IsLoading() && controller_.GetPendingEntry())
- controller_.DiscardPendingEntry();
+ controller_.DiscardPendingEntry(false);
// Update the URL display.
NotifyNavigationStateChanged(INVALIDATE_TYPE_URL);

Powered by Google App Engine
This is Rietveld 408576698