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

chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

Index: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
diff --git a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
index fd8b196f376443176b1b5d0926e14f2624a62292..8a7a3d87c99a93f5461eb441d8af7788f0bfda42 100644
--- a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
+++ b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
@@ -42,6 +42,7 @@
 #include "content/public/browser/resource_context.h"
 #include "content/public/browser/resource_dispatcher_host.h"
 #include "content/public/browser/resource_request_info.h"
+#include "content/public/browser/service_worker_context.h"
 #include "content/public/browser/stream_info.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/resource_response.h"
@@ -271,6 +272,9 @@ ChromeResourceDispatcherHostDelegate::ChromeResourceDispatcherHostDelegate(
       user_script_listener_(new extensions::UserScriptListener()),
 #endif
       prerender_tracker_(prerender_tracker) {
+  content::ServiceWorkerContext::AddExcludedHeadersForFetchEvent(

[Lei Zhang, 2014/10/29 21:47:21]

This looks like essentially a static function call. Why do we have to do it
every time a ChromeResourceDispatcherHostDelegate gets created?

[horo, 2014/10/30 07:54:34]

I think ChromeResourceDispatcherHostDelegate is created only once.
But it is better to call this on IO thread.
So I moved this to IOThread::InitAsync.

[Lei Zhang, 2014/10/30 08:05:17]

In that case, please as a chrome/browser/io_thread.cc OWNER to review. They know
the code much better than me.

+      variations::VariationsHttpHeaderProvider::GetInstance()
+          ->GetVariationHeaderNames());
 }
 
 ChromeResourceDispatcherHostDelegate::~ChromeResourceDispatcherHostDelegate() {