Transmit a X-Chrome-UMA-Enabled bit to Google domains from clients that have UMA enabled.

chrome/browser/net/chrome_network_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/net/chrome_network_delegate.h"
 
 #include "base/logging.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/content_settings/cookie_settings.h"
 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
 #include "chrome/browser/custom_handlers/protocol_handler_registry.h"
 #include "chrome/browser/extensions/api/proxy/proxy_api.h"
 #include "chrome/browser/extensions/api/web_request/web_request_api.h"
 #include "chrome/browser/extensions/extension_event_router_forwarder.h"
 #include "chrome/browser/extensions/extension_info_map.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
+#include "chrome/browser/google/google_util.h"
+#include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/browser/prefs/pref_member.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/task_manager/task_manager.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/resource_request_info.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/net_errors.h"
 #include "net/base/net_log.h"
@@ skipping 77 matching lines @@
 }
 
 }  // namespace
 
 ChromeNetworkDelegate::ChromeNetworkDelegate(
     ExtensionEventRouterForwarder* event_router,
     ExtensionInfoMap* extension_info_map,
     const policy::URLBlacklistManager* url_blacklist_manager,
     void* profile,
     CookieSettings* cookie_settings,
+    bool is_incognito,
     BooleanPrefMember* enable_referrers)
     : event_router_(event_router),
       profile_(profile),
       cookie_settings_(cookie_settings),
+      is_incognito_(is_incognito),
       extension_info_map_(extension_info_map),
       enable_referrers_(enable_referrers),
       url_blacklist_manager_(url_blacklist_manager) {
   DCHECK(event_router);
   DCHECK(enable_referrers);
   DCHECK(!profile || cookie_settings);
 }
 
 ChromeNetworkDelegate::~ChromeNetworkDelegate() {}
 
@@ skipping 30 matching lines @@
   if (!enable_referrers_->GetValue())
     request->set_referrer(std::string());
   return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeRequest(
       profile_, extension_info_map_.get(), request, callback, new_url);
 }
 
 int ChromeNetworkDelegate::OnBeforeSendHeaders(
     net::URLRequest* request,
     const net::CompletionCallback& callback,
     net::HttpRequestHeaders* headers) {
+  // Attach any applicable Chrome metrics headers. This is a best-effort attempt
+  // where failure will not cause OnBeforeSendHeaders to back out.
+  AddChromeMetricsStateHeader(request, headers);
+
   return ExtensionWebRequestEventRouter::GetInstance()->OnBeforeSendHeaders(
       profile_, extension_info_map_.get(), request, callback, headers);
 }
 
 void ChromeNetworkDelegate::OnSendHeaders(
     net::URLRequest* request,
     const net::HttpRequestHeaders& headers) {
   ExtensionWebRequestEventRouter::GetInstance()->OnSendHeaders(
       profile_, extension_info_map_.get(), request, headers);
 }
@@ skipping 119 matching lines @@
     BrowserThread::PostTask(
         BrowserThread::UI, FROM_HERE,
         base::Bind(&TabSpecificContentSettings::CookieChanged,
                    render_process_id, render_view_id,
                    request->url(), request->first_party_for_cookies(),
                    cookie_line, *options, !allow));
   }
 
   return allow;
 }
+
+void ChromeNetworkDelegate::AddChromeMetricsStateHeader(
+    net::URLRequest* request,
+    net::HttpRequestHeaders* headers) {
+  // Note our criteria for attaching Chrome experiment headers:
+  // 1. We only transmit to *.google.<TLD> domains. NOTE that this use of
+  //    google_util helpers to check this does not guarantee that the URL is
+  //    Google-owned, only that it is of the form *.google.<TLD>. In the future
+  //    we may choose to reinforce this check.
+  // 2. We must verify that we are transmitting for a profile network context
+  //    and not the system context. We do this by checking |profile_|, as it is
+  //    NULL when we're in the system context.
+  // 3. We only transmit for non Off The Record profiles.

[Ilya Sherman, 2012/04/17 20:55:39]

nit: s/Off The Record/Incognito

[SteveT, 2012/04/19 18:38:01]

Done.

+  // 4. For the X-Chrome-UMA-Enabled bit, we only set it if UMA is in fact
+  //    enabled for this install of Chrome.
+  if (google_util::IsGoogleDomainUrl(request->url().spec(),
+                                     google_util::ALLOW_SUBDOMAIN) &&
+      profile_ && !is_incognito_ &&
+      MetricsServiceHelper::IsMetricsReportingEnabled()) {

[battre, 2012/04/17 20:33:02]

Here we still have a threading problem. The MetricServiceHelper accesses the
PrefService which lives on the UI thread. You can use a PrefMember.

[SteveT, 2012/04/17 23:26:44]

Ah, yes. I see that we do something similar with the enable_referrers PrefMember
here. You're suggesting the same approach, but for the Metrics-Enabled pref?

We're sort of circumventing MetricServiceHelper::IsMetricsReportingEnabled...
but it looks like we have no choice since we're on the IO thread here.  Ilya can
let us know if using the PrefMember is okay or not.

[Ilya Sherman, 2012/04/17 23:33:13]

I think it's probably ok to use a PrefMember, and add a comment to the
MetricsService file documenting that this class implicitly relies on reading
that pref.  There's a slightly higher chance that the code will drift to be out
of sync, but I think it's still an ok compromise.

[battre, 2012/04/18 17:05:05]

We could probably also put the PrefMember in MetricsServiceHelper. I am not sure
which approach wold be cleaner.

[SteveT, 2012/04/19 18:38:01]

Turns out letting the PrefMember live in ProfileIOData makes the most sense. It
can be appropriately initialized on the UI thread like the enable_referrers
memeber.

willchan@ will be on the review to make sure our changes to Profile*IOData are
satisfactory.

+    headers->SetHeader("X-Chrome-UMA-Enabled", "1");
+  }
+}