Implement ServiceWorkerVersion::SendMessage() (for dispatching events etc)

content/renderer/service_worker/service_worker_script_context.h

Index: content/renderer/service_worker/service_worker_script_context.h
diff --git a/content/renderer/service_worker/service_worker_script_context.h b/content/renderer/service_worker/service_worker_script_context.h
index 86f1d8e5edc5fc390b4dc6f1a36f3b5106559904..9bed978fc186a46727cb0b47807ce76db2204437 100644
--- a/content/renderer/service_worker/service_worker_script_context.h
+++ b/content/renderer/service_worker/service_worker_script_context.h
@@ -31,11 +31,11 @@ class ServiceWorkerScriptContext {
       blink::WebServiceWorkerContextProxy* proxy);
   ~ServiceWorkerScriptContext();
 
-  void OnMessageReceived(const IPC::Message& message);
+  void OnMessageReceived(int request_id, const IPC::Message& message);

[alecflett, 2014/02/04 20:53:24]

I'm starting to think this should return a bool (for 'handled') and propagate
that as far as it can go - giving us a backstop to catch bad messages (i.e.
security checks)

[kinuko, 2014/02/04 22:50:24]

Yup, I had the same thought, that'll be addressed in my next patch

(my local patch once had it but I removed it before uploading since I didn't
want to make this patch bigger)

 
  private:
   // Send message back to the browser.
-  void Send(const IPC::Message& message);
+  void Send(int request_id, const IPC::Message& message);
 
   void OnFetchEvent(const ServiceWorkerFetchRequest& request);
 
@@ -45,6 +45,9 @@ class ServiceWorkerScriptContext {
   // Not owned; this object is destroyed when proxy_ becomes invalid.
   blink::WebServiceWorkerContextProxy* proxy_;
 
+  // Used by message handlers.
+  int current_request_id_;
+
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerScriptContext);
 };