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

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

Issue 243493002: Clear failed browser-initiated navigations if blank tab is modified. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix comment Created 6 years, 8 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
« no previous file with comments | « content/browser/frame_host/navigation_controller_impl_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 0cc4374543015572b58996ebc14436a0d4edcb29..39823da9e30dbfff5ed02311b67e8ff717252d13 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -3365,6 +3365,12 @@ void WebContentsImpl::DidDisownOpener(RenderViewHost* rvh) {
}
void WebContentsImpl::DidAccessInitialDocument() {
+ // We may have left a failed browser-initiated navigation in the address bar
+ // 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();
+
// Update the URL display.
NotifyNavigationStateChanged(content::INVALIDATE_TYPE_URL);
}
« no previous file with comments | « content/browser/frame_host/navigation_controller_impl_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698