Introduces DevToolsManagerDelegate.

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 "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "url/gurl.h"
 
 namespace content {
 
 // Represents the per-StoragePartition ServiceWorker data.  Must be used from
 // the UI thread.
 class ServiceWorkerContext {
  public:
+  class ServiceWorkerInfo {

[michaeln, 2014/05/01 02:52:07]

naming nit: RunningServiceWorkerInfo? I can easily imagine a different kind of
'info' making it into the this api.

[horo, 2014/05/01 04:52:32]

Done.

+   public:
+    ServiceWorkerInfo(int worker_process_id,
+                      int worker_devtools_agent_route_id,
+                      const GURL& scope,
+                      const GURL& url)
+        : worker_process_id(worker_process_id),
+          worker_devtools_agent_route_id(worker_devtools_agent_route_id),
+          scope(scope),
+          url(url) {}
+    int worker_process_id;
+    int worker_devtools_agent_route_id;
+    GURL scope;
+    GURL url;
+  };
+
   // 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<
+      ServiceWorkerInfo>& registrations)> GetRunningServiceWorkerInfoCallback;
+
   // 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.
   //  * Something unexpected goes wrong, like a renderer crash or a full disk.
   virtual void RegisterServiceWorker(const Scope& pattern,
                                      const GURL& script_url,
                                      const ResultCallback& callback) = 0;
 
   // Equivalent to calling navigator.serviceWorker.unregister(pattern) from a
   // renderer in |source_process_id|, 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.
   //
   // Unregistration can fail if:
   //  * No Service Worker was registered for |pattern|.
   //  * Something unexpected goes wrong, like a renderer crash.
   virtual void UnregisterServiceWorker(const Scope& pattern,
                                        int source_process_id,
                                        const ResultCallback& callback) = 0;
 
   // TODO(jyasskin): Provide a way to SendMessage to a Scope.
 
+  virtual void AddStatusChangeCallback(
+      const base::Callback<void(void)>& callback) = 0;
+
+  virtual void RemoveStatusChangeCallback(
+      const base::Callback<void(void)>& callback) = 0;

[michaeln, 2014/05/01 02:52:07]

Identifying what to remove by callback ref seems odd somehow? Two callbacks
produced with the same inputs to two different calls to base::Bind are not
equal. Also this could definitely use some more specificity about what changed.

These kind of notifications might be better served with Listener style
delegation. I think vincent's api work is also headed toward a
ServiceWorkerContextClient or Listener sort of class in the content api.

[horo, 2014/05/01 04:52:32]

Changed to use StatusChangeObserver.

+
+  virtual void GetRunningServiceWorkerInfo(
+      const GetRunningServiceWorkerInfoCallback& callback) = 0;
+
  protected:
   ServiceWorkerContext() {}
   virtual ~ServiceWorkerContext() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_SERVICE_WORKER_CONTEXT_H_