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

Unified Diff: net/url_request/url_request.h

Issue 12569007: Remove URL fragment from referrer HTTP header when opening link using "Open Link in New Tab" option. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address code review comments Created 7 years, 9 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: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index 5580b63eef8925633345c155f356b403b120a26d..416d1f8597ea23b3bd89f687b2d974c6f1d7d110 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -370,9 +370,8 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
// URL will not be sent as the referrer for a HTTP request). The referrer
// may only be changed before Start() is called.
const std::string& referrer() const { return referrer_; }
- void set_referrer(const std::string& referrer);
- // Returns the referrer header with potential username and password removed.
- GURL GetSanitizedReferrer() const;
+ // Referrer is sanitized to remove URL fragment, user name and password
pauljensen 2013/03/15 13:37:45 Please add period.
mef 2013/03/15 16:36:42 Done.
+ void SetReferrer(const std::string& referrer);
// The referrer policy to apply when updating the referrer during redirects.
// The referrer policy may only be changed before Start() is called.

Powered by Google App Engine
This is Rietveld 408576698