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

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

Issue 9978015: Make browser_handles_top_level_requests synchronous. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ExternalTabContainer::ShouldIgnoreNavigation returns true if is_content_initiated Created 8 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
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 d7b9faee85cc3b6c13e9dfa5f29d155ca32b2426..f986521d2a1aa088e68fddf8b0a68ce7ca37e364 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -2326,6 +2326,23 @@ void WebContentsImpl::RequestTransferURL(
}
}
+bool WebContentsImpl::ShouldIgnoreNavigation(
+ const GURL& url,
+ const content::Referrer& referrer,
+ WindowOpenDisposition disposition,
+ int64 source_frame_id,
+ bool is_content_initiated) {
+ if (!delegate_)
+ return false;
+ content::PageTransition transition_type = content::PAGE_TRANSITION_LINK;
+ NavigationEntry* active_navigation_entry = controller_.GetActiveEntry();
+ if (active_navigation_entry) {
+ transition_type = active_navigation_entry->GetTransitionType();
+ }
+ return delegate_->ShouldIgnoreNavigation(
+ this, url, referrer, disposition, is_content_initiated, transition_type);
+}
+
void WebContentsImpl::RunJavaScriptMessage(
RenderViewHost* rvh,
const string16& message,

Powered by Google App Engine
This is Rietveld 408576698