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

Unified Diff: chrome/browser/extensions/webstore_inline_installer.cc

Issue 8568037: Set referrer for webstore inline install requests, so we can see what the top pages that use it are (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/webstore_inline_installer.cc
diff --git a/chrome/browser/extensions/webstore_inline_installer.cc b/chrome/browser/extensions/webstore_inline_installer.cc
index a3af2c638f2366c9da4c978df3900781a60c1d23..a65936fd96ec7f32766cfdf366422c6d7359c570 100644
--- a/chrome/browser/extensions/webstore_inline_installer.cc
+++ b/chrome/browser/extensions/webstore_inline_installer.cc
@@ -175,6 +175,10 @@ void WebstoreInlineInstaller::BeginInstall() {
tab_contents()->browser_context());
webstore_data_url_fetcher_->SetRequestContext(
profile->GetRequestContext());
+ // Use the requesting page as the referrer both since that is more correct
+ // (it is the page that caused this request to happen) and so that we can
+ // track top sites that trigger inline install requests.
+ webstore_data_url_fetcher_->SetReferrer(requestor_url_.spec());
webstore_data_url_fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES |
net::LOAD_DO_NOT_SAVE_COOKIES |
net::LOAD_DISABLE_CACHE);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698