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

Side by Side Diff: chrome/browser/net/chrome_network_delegate.cc

Issue 10107009: Revert 132479 - Transmit a X-Chrome-UMA-Enabled bit to Google domains from clients that have UMA en… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/net/chrome_network_delegate.h" 5 #include "chrome/browser/net/chrome_network_delegate.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "chrome/browser/browser_process.h" 8 #include "chrome/browser/browser_process.h"
9 #include "chrome/browser/content_settings/cookie_settings.h" 9 #include "chrome/browser/content_settings/cookie_settings.h"
10 #include "chrome/browser/content_settings/tab_specific_content_settings.h" 10 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
11 #include "chrome/browser/custom_handlers/protocol_handler_registry.h" 11 #include "chrome/browser/custom_handlers/protocol_handler_registry.h"
12 #include "chrome/browser/extensions/api/proxy/proxy_api.h" 12 #include "chrome/browser/extensions/api/proxy/proxy_api.h"
13 #include "chrome/browser/extensions/api/web_request/web_request_api.h" 13 #include "chrome/browser/extensions/api/web_request/web_request_api.h"
14 #include "chrome/browser/extensions/extension_event_router_forwarder.h" 14 #include "chrome/browser/extensions/extension_event_router_forwarder.h"
15 #include "chrome/browser/extensions/extension_info_map.h" 15 #include "chrome/browser/extensions/extension_info_map.h"
16 #include "chrome/browser/extensions/extension_process_manager.h" 16 #include "chrome/browser/extensions/extension_process_manager.h"
17 #include "chrome/browser/google/google_util.h"
18 #include "chrome/browser/metrics/metrics_service.h"
19 #include "chrome/browser/prefs/pref_member.h" 17 #include "chrome/browser/prefs/pref_member.h"
20 #include "chrome/browser/profiles/profile_manager.h" 18 #include "chrome/browser/profiles/profile_manager.h"
21 #include "chrome/browser/task_manager/task_manager.h" 19 #include "chrome/browser/task_manager/task_manager.h"
22 #include "chrome/common/pref_names.h" 20 #include "chrome/common/pref_names.h"
23 #include "content/public/browser/browser_thread.h" 21 #include "content/public/browser/browser_thread.h"
24 #include "content/public/browser/render_view_host.h" 22 #include "content/public/browser/render_view_host.h"
25 #include "content/public/browser/resource_request_info.h" 23 #include "content/public/browser/resource_request_info.h"
26 #include "net/base/host_port_pair.h" 24 #include "net/base/host_port_pair.h"
27 #include "net/base/net_errors.h" 25 #include "net/base/net_errors.h"
28 #include "net/base/net_log.h" 26 #include "net/base/net_log.h"
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 if (!enable_referrers_->GetValue()) 158 if (!enable_referrers_->GetValue())
161 request->set_referrer(std::string()); 159 request->set_referrer(std::string());
162 return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeRequest( 160 return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeRequest(
163 profile_, extension_info_map_.get(), request, callback, new_url); 161 profile_, extension_info_map_.get(), request, callback, new_url);
164 } 162 }
165 163
166 int ChromeNetworkDelegate::OnBeforeSendHeaders( 164 int ChromeNetworkDelegate::OnBeforeSendHeaders(
167 net::URLRequest* request, 165 net::URLRequest* request,
168 const net::CompletionCallback& callback, 166 const net::CompletionCallback& callback,
169 net::HttpRequestHeaders* headers) { 167 net::HttpRequestHeaders* headers) {
170 // Attach any applicable Chrome metrics headers. This is a best-effort attempt
171 // where failure will not cause OnBeforeSendHeaders to back out.
172 AddChromeMetricsStateHeader(request, headers);
173
174 return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeSendHeaders( 168 return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeSendHeaders(
175 profile_, extension_info_map_.get(), request, callback, headers); 169 profile_, extension_info_map_.get(), request, callback, headers);
176 } 170 }
177 171
178 void ChromeNetworkDelegate::OnSendHeaders( 172 void ChromeNetworkDelegate::OnSendHeaders(
179 net::URLRequest* request, 173 net::URLRequest* request,
180 const net::HttpRequestHeaders& headers) { 174 const net::HttpRequestHeaders& headers) {
181 ExtensionWebRequestEventRouter::GetInstance()->OnSendHeaders( 175 ExtensionWebRequestEventRouter::GetInstance()->OnSendHeaders(
182 profile_, extension_info_map_.get(), request, headers); 176 profile_, extension_info_map_.get(), request, headers);
183 } 177 }
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
303 BrowserThread::PostTask( 297 BrowserThread::PostTask(
304 BrowserThread::UI, FROM_HERE, 298 BrowserThread::UI, FROM_HERE,
305 base::Bind(&TabSpecificContentSettings::CookieChanged, 299 base::Bind(&TabSpecificContentSettings::CookieChanged,
306 render_process_id, render_view_id, 300 render_process_id, render_view_id,
307 request->url(), request->first_party_for_cookies(), 301 request->url(), request->first_party_for_cookies(),
308 cookie_line, *options, !allow)); 302 cookie_line, *options, !allow));
309 } 303 }
310 304
311 return allow; 305 return allow;
312 } 306 }
313
314 void ChromeNetworkDelegate::AddChromeMetricsStateHeader(
315 net::URLRequest* request,
316 net::HttpRequestHeaders* headers) {
317 // Note our criteria for attaching Chrome experiment headers:
318 // 1. We only transmit to *.google.<TLD> domains. NOTE that this use of
319 // google_util helpers to check this does not guarantee that the URL is
320 // Google-owned, only that it is of the form *.google.<TLD>. In the future
321 // we may choose to reinforce this check.
322 // 2. We must verify that the transmitting profile is not off the record.
323 // 3. For the X-Chrome-UMA-Enabled bit, we only set it if UMA is in fact
324 // enabled for this install of Chrome.
325 Profile* profile_instance = reinterpret_cast<Profile*>(profile_);
326 if (google_util::IsGoogleDomainUrl(request->url().spec(),
327 google_util::ALLOW_SUBDOMAIN) &&
328 profile_instance && !profile_instance->IsOffTheRecord() &&
329 MetricsServiceHelper::IsMetricsReportingEnabled()) {
330 headers->SetHeader("X-Chrome-UMA-Enabled", "1");
331 }
332 }
OLDNEW
« no previous file with comments | « chrome/browser/net/chrome_network_delegate.h ('k') | chrome/browser/protector/protector_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698