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

chrome/browser/devtools/devtools_network_transaction.cc

 // Copyright 2014 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/devtools/devtools_network_transaction.h"
 
 #include "chrome/browser/devtools/devtools_network_controller.h"
 #include "chrome/browser/devtools/devtools_network_interceptor.h"
 #include "net/base/net_errors.h"
 #include "net/base/upload_progress.h"
 #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,
     scoped_ptr<net::HttpTransaction> network_transaction)
     : controller_(controller),
       network_transaction_(network_transaction.Pass()),
       request_(NULL),
       failed_(false),
       throttled_byte_count_(0),
       callback_type_(NONE),
@@ skipping 235 matching lines @@
 }
 
 void DevToolsNetworkTransaction::FireThrottledCallback() {
   DCHECK(!callback_.is_null());
   DCHECK(callback_type_ == READ || callback_type_ == START);
   net::CompletionCallback callback = callback_;
   callback_.Reset();
   callback_type_ = NONE;
   callback.Run(throttled_result_);
 }