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

Unified Diff: content/browser/tab_contents/tab_contents.cc

Issue 8224023: Don't show URL for pending new navigations initiated by the renderer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix merge conflicts. Created 9 years, 2 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/tab_contents/tab_contents.cc
diff --git a/content/browser/tab_contents/tab_contents.cc b/content/browser/tab_contents/tab_contents.cc
index 8557a2be67733237d24b243efa0df86a554453d9..90d5eab45314245d7818909b8a3a663a6d044529 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -536,7 +536,8 @@ TabContents* TabContents::OpenURL(const GURL& url,
const GURL& referrer,
WindowOpenDisposition disposition,
content::PageTransition transition) {
- return OpenURL(OpenURLParams(url, referrer, disposition, transition));
+ return OpenURL(OpenURLParams(url, referrer, disposition, transition,
+ false));
}
TabContents* TabContents::OpenURL(const OpenURLParams& params) {
@@ -1685,8 +1686,9 @@ void TabContents::RequestOpenURL(const GURL& url,
render_manager_.web_ui()->link_transition_type());
transition_type = render_manager_.web_ui()->link_transition_type();
} else {
- new_contents = OpenURL(
- url, referrer, disposition, content::PAGE_TRANSITION_LINK);
+ new_contents = OpenURL(OpenURLParams(
+ url, referrer, disposition, content::PAGE_TRANSITION_LINK,
+ true /* is_renderer_initiated */));
}
if (new_contents) {
// Notify observers.

Powered by Google App Engine
This is Rietveld 408576698