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

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

Issue 8801006: Make the transfer request code path use full referrers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updates Created 9 years 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 3dae106f371be8d28e8f9351b234c62a08a14bf3..1235c9a99b942b2a3155c2c994fa3492626de47a 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -1757,12 +1757,12 @@ void TabContents::RequestOpenURL(const GURL& url,
int64 source_frame_id) {
// Delegate to RequestTransferURL because this is just the generic
// case where |old_request_id| is empty.
- RequestTransferURL(url, referrer.url, disposition, source_frame_id,
+ RequestTransferURL(url, referrer, disposition, source_frame_id,
GlobalRequestID());
}
void TabContents::RequestTransferURL(const GURL& url,
- const GURL& referrer,
+ const content::Referrer& referrer,
WindowOpenDisposition disposition,
int64 source_frame_id,
const GlobalRequestID& old_request_id) {
@@ -1784,7 +1784,7 @@ void TabContents::RequestTransferURL(const GURL& url,
new_contents = OpenURL(params);
transition_type = render_manager_.web_ui()->link_transition_type();
} else {
- OpenURLParams params(url, referrer, disposition,
+ OpenURLParams params(url, referrer.url, disposition,
content::PAGE_TRANSITION_LINK, true /* is_renderer_initiated */);
params.transferred_global_request_id = old_request_id;
new_contents = OpenURL(params);
@@ -1794,7 +1794,7 @@ void TabContents::RequestTransferURL(const GURL& url,
FOR_EACH_OBSERVER(TabContentsObserver, observers_,
DidOpenRequestedURL(new_contents,
url,
- referrer,
+ referrer.url,
disposition,
transition_type,
source_frame_id));

Powered by Google App Engine
This is Rietveld 408576698