Flush out initial [un]registerServiceWorker roundtrip

content/browser/service_worker/service_worker_dispatcher_host.h

Index: content/browser/service_worker/service_worker_dispatcher_host.h
diff --git a/content/browser/service_worker/service_worker_dispatcher_host.h b/content/browser/service_worker/service_worker_dispatcher_host.h
index 36927819f69f0d8e0a879252f2a5a42fee133bae..d809f89b8d24e2d0ca00c0f3c59994377feb32c9 100644
--- a/content/browser/service_worker/service_worker_dispatcher_host.h
+++ b/content/browser/service_worker/service_worker_dispatcher_host.h
@@ -15,7 +15,8 @@ class ServiceWorkerContext;
 
 class ServiceWorkerDispatcherHost : public BrowserMessageFilter {
  public:
-  explicit ServiceWorkerDispatcherHost(ServiceWorkerContext* context);
+  ServiceWorkerDispatcherHost(int render_process_id,
+                              ServiceWorkerContext* context);
 
   // BrowserIOMessageFilter implementation
   virtual bool OnMessageReceived(const IPC::Message& message,
@@ -26,12 +27,14 @@ class ServiceWorkerDispatcherHost : public BrowserMessageFilter {
 
  private:
   // IPC Message handlers
-
-  void OnRegisterServiceWorker(int32 registry_id,
-                            const string16& scope,
-                            const GURL& script_url);
-  void OnUnregisterServiceWorker(int32 registry_id, const string16& scope);
-
+  void OnRegisterServiceWorker(int32 thread_id,
+                               int32 request_id,
+                               const GURL& scope,
+                               const GURL& script_url);
+  void OnUnregisterServiceWorker(int32 thread_id,
+                                 int32 request_id,
+                                 const GURL& scope);
+  int render_process_id_ ALLOW_UNUSED;

[jam, 2013/10/28 17:26:55]

nit: blank line above this

why are you adding ALLOW_UNUSED? this isn't something that's widespread in the
code

[alecflett, 2013/10/28 21:11:43]

not right this moment, but in a future patch. An earlier iteration of this patch
included a security check, but we're holding off on that until a little later
(see the other TODO's in this patch)

   scoped_refptr<ServiceWorkerContext> context_;
 };