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

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

Issue 8784006: Replace the GURL referrer field of OpenURLParams with a content::Referrer (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
« no previous file with comments | « content/browser/tab_contents/tab_contents.h ('k') | content/browser/tab_contents/tab_contents_delegate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..489e38d72da7789e4ee84d51e12de8178972aeca 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -580,8 +580,8 @@ TabContents* TabContents::OpenURL(const GURL& url,
content::PageTransition transition) {
// For specifying a referrer, use the version of OpenURL taking OpenURLParams.
DCHECK(referrer.is_empty());
- return OpenURL(OpenURLParams(url, referrer, disposition, transition,
- false));
+ return OpenURL(OpenURLParams(url, content::Referrer(), disposition,
+ transition, false));
}
TabContents* TabContents::OpenURL(const OpenURLParams& params) {
@@ -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) {
@@ -1777,7 +1777,7 @@ void TabContents::RequestTransferURL(const GURL& url,
// want web sites to see a referrer of "chrome://blah" (and some
// chrome: URLs might have search terms or other stuff we don't want to
// send to the site), so we send no referrer.
- OpenURLParams params(url, GURL(), disposition,
+ OpenURLParams params(url, content::Referrer(), disposition,
render_manager_.web_ui()->link_transition_type(),
false /* is_renderer_initiated */);
params.transferred_global_request_id = old_request_id;
« no previous file with comments | « content/browser/tab_contents/tab_contents.h ('k') | content/browser/tab_contents/tab_contents_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698