Flush out initial [un]registerServiceWorker roundtrip

content/child/service_worker/service_worker_dispatcher.h

+// Copyright 2013 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_CHILD_SERVICE_WORKER_SERVICE_WORKER_DISPATCHER_H_
+#define CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_DISPATCHER_H_
+
+#include "base/id_map.h"
+#include "base/memory/ref_counted.h"
+#include "base/strings/string16.h"
+#include "third_party/WebKit/public/platform/WebServiceWorkerProvider.h"
+#include "webkit/child/worker_task_runner.h"
+
+class GURL;
+
+namespace WebKit {
+class WebURL;
+}
+
+namespace IPC {
+class Message;
+}
+
+namespace content {
+class ThreadSafeSender;
+class ServiceWorkerMessageFilter;
+class WebServiceWorkerImpl;
+
+class ServiceWorkerDispatcher : public webkit_glue::WorkerTaskRunner::Observer {
+ public:
+  ServiceWorkerDispatcher(ThreadSafeSender* thread_safe_sender);
+  virtual ~ServiceWorkerDispatcher();
+
+  // webkit_glue::WorkerTaskRunner::Observer implementation.
+  virtual void OnWorkerRunLoopStopped() OVERRIDE;

[michaeln, 2013/10/08 23:08:38]

nit: could this method be private

[alecflett, 2013/10/09 01:23:47]

Done.

+
+  void OnMessageReceived(const IPC::Message& msg);
+  bool Send(IPC::Message* msg);
+
+  void RegisterServiceWorker(
+      const WebKit::WebURL& origin,
+      const string16& pattern,
+      const GURL& script_url,
+      WebKit::WebServiceWorkerProvider::WebServiceWorkerCallbacks* callbacks);
+  void UnregisterServiceWorker(
+      const WebKit::WebURL& origin,
+      const string16& pattern,
+      WebKit::WebServiceWorkerProvider::WebServiceWorkerCallbacks* callbacks);
+
+  // |thread_safe_sender| needs to be passed in because if the call leads to
+  // construction it will be needed.
+  static ServiceWorkerDispatcher* ThreadSpecificInstance(
+      ThreadSafeSender* thread_safe_sender);
+
+ private:
+  void OnServiceWorkerRegistered(int32 thread_id,
+                                 int32 request_id,

[michaeln, 2013/10/08 23:08:38]

here its called request_id... that's the name i like ;)

[alecflett, 2013/10/09 01:23:47]

Sorry I'll try to make these a little more consistent.

+                                 int32 worker_id);

[michaeln, 2013/10/08 23:08:38]

maybe expand this to service_worker_id so its less easily confused with a
webcore worker thread id of some kind (although i think we'll have lots of
confusion along those lines no matter what we do given our chosen name for the
uniquely and persistently identified "ServiceWorker" thing).

[alecflett, 2013/10/09 01:23:47]

Done.

+  void OnServiceWorkerUnregistered(int32 thread_id,
+                                   int32 request_id,
+                                   int32 worker_id);
+
+  IDMap<WebKit::WebServiceWorkerProvider::WebServiceWorkerCallbacks,
+        IDMapOwnPointer> pending_callbacks_;
+
+  scoped_refptr<ThreadSafeSender> thread_safe_sender_;
+
+  DISALLOW_COPY_AND_ASSIGN(ServiceWorkerDispatcher);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_DISPATCHER_H_