[ServiceWorker] Don't send the UMA related headers to the ServiceWorker.

content/browser/service_worker/service_worker_url_request_job.cc

Index: content/browser/service_worker/service_worker_url_request_job.cc
diff --git a/content/browser/service_worker/service_worker_url_request_job.cc b/content/browser/service_worker/service_worker_url_request_job.cc
index a041426213c33d26ee4ef0b1e841fa9517cfe23a..a926622387da0bc9af0c9825a9ce58ec35903cb1 100644
--- a/content/browser/service_worker/service_worker_url_request_job.cc
+++ b/content/browser/service_worker/service_worker_url_request_job.cc
@@ -39,6 +39,10 @@ const char kDevToolsRequestInitiator[] = "X-DevTools-Request-Initiator";
 const char kDevToolsEmulateNetworkConditionsClientId[] =
     "X-DevTools-Emulate-Network-Conditions-Client-Id";
 
+// Keep in sync with variations_http_header_provider.cc.

[michaeln, 2014/10/22 22:19:40]

Having to manually keep these files in sysnc is probably not a good idea.

[Alexei Svitkine (slow), 2014/10/24 15:35:48]

Agreed.

I think we probably want to have a IsVariationsHeader(name) method or similar on
VariationsHttpHeaderProvider and call this from here (probably through an
interface, since this code is content likely shouldn't depend on the variations
component).

[michaeln, 2014/10/24 19:49:50]

The content lib cannot depend on variations component, but i believe there's no
issue with variations_http_header_provider component depending on the content
lib. This is why i was suggesting an content/public interface for the
variations_http_header_provider to invoke that would inform the sw system about
which to exclude. I had in mind a one-time call that would add values to a
global map.

[horo, 2014/10/27 14:15:40]

Ah. I didn't notice that there is 'variations_http_provider' target in
variations.gypi.

I uploaded Patch Set 2.
Please review this.

+const char kChromeUMAEnabled[] = "X-Chrome-UMA-Enabled";
+const char kClientData[] = "X-Client-Data";
+
 }  // namespace
 
 ServiceWorkerURLRequestJob::ServiceWorkerURLRequestJob(
@@ -300,7 +304,9 @@ ServiceWorkerURLRequestJob::CreateFetchRequest() {
   const net::HttpRequestHeaders& headers = request_->extra_request_headers();
   for (net::HttpRequestHeaders::Iterator it(headers); it.GetNext();) {
     if (it.name() == kDevToolsRequestInitiator ||
-        it.name() == kDevToolsEmulateNetworkConditionsClientId) {
+        it.name() == kDevToolsEmulateNetworkConditionsClientId ||
+        it.name() == kChromeUMAEnabled ||
+        it.name() == kClientData) {
       continue;
     }
     request->headers[it.name()] = it.value();