Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(992)

Unified Diff: chrome/browser/devtools/devtools_network_transaction_factory.cc

Issue 666973003: [ServiceWorker] Don't send the UMA related headers to the ServiceWorker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use set Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/devtools/devtools_network_transaction_factory.cc
diff --git a/chrome/browser/devtools/devtools_network_transaction_factory.cc b/chrome/browser/devtools/devtools_network_transaction_factory.cc
index 3fd8b84b4caa42b02c68068b2423d60d282b6d71..09b8ddc88a261321f8ed74d9ee4b6e1fd04fabb2 100644
--- a/chrome/browser/devtools/devtools_network_transaction_factory.cc
+++ b/chrome/browser/devtools/devtools_network_transaction_factory.cc
@@ -4,8 +4,12 @@
#include "chrome/browser/devtools/devtools_network_transaction_factory.h"
+#include <set>
+#include <string>
+
#include "chrome/browser/devtools/devtools_network_controller.h"
#include "chrome/browser/devtools/devtools_network_transaction.h"
+#include "content/public/browser/service_worker_context.h"
#include "net/base/net_errors.h"
#include "net/http/http_network_layer.h"
#include "net/http/http_network_transaction.h"
@@ -15,6 +19,11 @@ DevToolsNetworkTransactionFactory::DevToolsNetworkTransactionFactory(
net::HttpNetworkSession* session)
: controller_(controller),
network_layer_(new net::HttpNetworkLayer(session)) {
+ std::set<std::string> headers;
+ headers.insert(DevToolsNetworkTransaction::kDevToolsRequestInitiator);
+ headers.insert(
+ DevToolsNetworkTransaction::kDevToolsEmulateNetworkConditionsClientId);
+ content::ServiceWorkerContext::AddExcludedHeadersForFetchEvent(headers);
}
DevToolsNetworkTransactionFactory::~DevToolsNetworkTransactionFactory() {

Powered by Google App Engine
This is Rietveld 408576698