[1/5 chromium] Shows the clients which are controlled by ServiceWorker in DevTools.

content/browser/service_worker/service_worker_utils.h

Index: content/browser/service_worker/service_worker_utils.h
diff --git a/content/browser/service_worker/service_worker_utils.h b/content/browser/service_worker/service_worker_utils.h
index c03f4f2dccdd99b0da60f00dac0ff4fbc191f7f3..f0d22362d6a821b0282b4955b6f0ebc41da4cd7e 100644
--- a/content/browser/service_worker/service_worker_utils.h
+++ b/content/browser/service_worker/service_worker_utils.h
@@ -6,6 +6,7 @@
 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_UTILS_H_
 
 #include "base/macros.h"
+#include "content/browser/service_worker/service_worker_provider_host.h"

[nhiroki, 2015/06/01 05:04:09]

nit: not necessary?

 #include "content/common/content_export.h"
 #include "content/common/service_worker/service_worker_status_code.h"
 #include "content/public/common/resource_type.h"
@@ -21,6 +22,8 @@ class ServiceWorkerUtils {
 
   // A helper for creating a do-nothing status callback.
   static void NoOpStatusCallback(ServiceWorkerStatusCode status) {}
+  // A helper for creating a do-nothing client info callback.
+  static void NoOpClientInfoCallback(const ServiceWorkerClientInfo&) {}

[nhiroki, 2015/06/01 05:04:09]

Can you move this into service_worker_context_wrapper.cc?

(NoOpStatusCallback is widely used among SW components, but this is not)

[horo, 2015/06/02 03:14:48]

Done.

[horo, 2015/06/02 03:14:48]

Done.

 
   // Returns true if |scope| matches |url|.
   CONTENT_EXPORT static bool ScopeMatches(const GURL& scope, const GURL& url);