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

content/public/browser/service_worker_context.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_
 #define CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_
 
+#include <set>

[no sievers, 2014/10/29 21:19:58]

set is unused

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

Done.

+#include <string>
+#include <vector>
+
 #include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "content/public/browser/service_worker_usage_info.h"
 #include "url/gurl.h"
 
 namespace content {
 
 // Represents the per-StoragePartition ServiceWorker data.  Must be used from
-// the IO thread.
+// the IO thread except for AddExcludedHeadersForFetchEvent.

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

With the code as is, do you mean to say "except for
AddExcludedHeadersForFetchEvent() which can be called from any thread" ?

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

Yes but the sentence "Must be used from the IO thread." was removed by
https://codereview.chromium.org/653633004.
So I removed this change.

 class ServiceWorkerContext {
  public:
   // https://rawgithub.com/slightlyoff/ServiceWorker/master/spec/service_worker/index.html#url-scope:
   // roughly, must be of the form "<origin>/<path>/*".
   typedef GURL Scope;
 
   typedef base::Callback<void(bool success)> ResultCallback;
 
   typedef base::Callback<void(const std::vector<ServiceWorkerUsageInfo>&
                                   usage_info)> GetUsageInfoCallback;
 
+  // Registers the header name which should not be passed to the ServiceWorker.
+  CONTENT_EXPORT static void AddExcludedHeadersForFetchEvent(
+      const std::vector<std::string>& header_names);
+
+  // Returns true if the header name should not be passed to the ServiceWorker.
+  static bool IsExcludedHeaderNameForFetchEvent(const std::string& header_name);
+
   // Equivalent to calling navigator.serviceWorker.register(script_url, {scope:
   // pattern}) from a renderer, except that |pattern| is an absolute URL instead
   // of relative to some current origin.  |callback| is passed true when the JS
   // promise is fulfilled or false when the JS promise is rejected.
   //
   // The registration can fail if:
   //  * |script_url| is on a different origin from |pattern|
   //  * Fetching |script_url| fails.
   //  * |script_url| fails to parse or its top-level execution fails.
   //    TODO: The error message for this needs to be available to developers.
@@ skipping 25 matching lines @@
   virtual void DeleteForOrigin(const GURL& origin_url) = 0;
 
  protected:
   ServiceWorkerContext() {}
   virtual ~ServiceWorkerContext() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_