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

chrome/browser/devtools/devtools_network_transaction.cc

Index: chrome/browser/devtools/devtools_network_transaction.cc
diff --git a/chrome/browser/devtools/devtools_network_transaction.cc b/chrome/browser/devtools/devtools_network_transaction.cc
index 44111d77aee7e38503bd7ed078137b21e0ca2397..6a0f1b578150a908f2ec61b75eda8bee1288dd46 100644
--- a/chrome/browser/devtools/devtools_network_transaction.cc
+++ b/chrome/browser/devtools/devtools_network_transaction.cc
@@ -11,16 +11,14 @@
 #include "net/http/http_network_transaction.h"
 #include "net/http/http_request_info.h"
 
-namespace {
-
 // Keep in sync with kDevToolsRequestInitiator and
 // kDevToolsEmulateNetworkConditionsClientId defined in
-// service_worker_url_request_job.cc and InspectorResourceAgent.cpp.
-const char kDevToolsRequestInitiator[] = "X-DevTools-Request-Initiator";
-const char kDevToolsEmulateNetworkConditionsClientId[] =
-    "X-DevTools-Emulate-Network-Conditions-Client-Id";
-
-}  // namespace
+// InspectorResourceAgent.cpp.
+const char DevToolsNetworkTransaction::kDevToolsRequestInitiator[] =
+    "X-DevTools-Request-Initiator";

[Alexei Svitkine (slow), 2014/10/27 15:09:00]

Why not move these to devtools_network_transaction_factory.cc where they're
actually used? Then you don't need to make them class members.

[horo, 2014/10/28 02:35:05]

They are used in DevToolsNetworkTransaction::ProcessRequest().
But DevToolsNetworkTransaction is created per every network transaction.
So I think we should call AddExcludedHeaderNameForFetchEvent() in the
constructor of DevToolsNetworkTransactionFactory which is created per profiles.

+const char
+    DevToolsNetworkTransaction::kDevToolsEmulateNetworkConditionsClientId[] =
+        "X-DevTools-Emulate-Network-Conditions-Client-Id";
 
 DevToolsNetworkTransaction::DevToolsNetworkTransaction(
     DevToolsNetworkController* controller,