[ServiceWorker] Mojofy InstallEvent of Service Worker

content/common/service_worker/service_worker_event_dispatcher.mojom

Index: content/common/service_worker/service_worker_event_dispatcher.mojom
diff --git a/content/common/service_worker/service_worker_event_dispatcher.mojom b/content/common/service_worker/service_worker_event_dispatcher.mojom
index 39eb5f400f4ad91876ee5a4736a3beac4fb9398f..134475ecb7e1e7695b248b405285d788c799bc6e 100644
--- a/content/common/service_worker/service_worker_event_dispatcher.mojom
+++ b/content/common/service_worker/service_worker_event_dispatcher.mojom
@@ -12,6 +12,7 @@ import "third_party/WebKit/public/platform/modules/background_sync/background_sy
 import "third_party/WebKit/public/platform/modules/fetch/fetch_api_request.mojom";
 import "third_party/WebKit/public/platform/modules/serviceworker/service_worker_event_status.mojom";
 import "url/mojo/origin.mojom";
+import "url/mojo/url.mojom";
 
 [Native]
 struct PlatformNotificationData;
@@ -53,6 +54,10 @@ struct ExtendableMessageEvent {
 // Renderer-side interface bound to ServiceWorkerContextClient for dispatching
 // events.
 interface ServiceWorkerEventDispatcher {
+  DispatchInstallEvent(associated InstallEventMethod method)
+      => (blink.mojom.ServiceWorkerEventStatus status,
+          bool has_fetch_handler,
+          mojo.common.mojom.Time dispatch_event_time);
   DispatchActivateEvent()
       => (blink.mojom.ServiceWorkerEventStatus status,
           mojo.common.mojom.Time dispatch_event_time);
@@ -110,3 +115,7 @@ interface ServiceWorkerEventDispatcher {
   // dispatch an event.
   Ping() => ();
 };
+
+interface InstallEventMethod {

[shimazu, 2017/04/05 05:01:27]

Could you rename it to ServiceWorkerInstallEventMethods?
It's in the namespace of 'content.mojom', so we should have the 'ServiceWorker'
prefix.

Also could you add some comment to explain this interface like the following?
--
// This interface is passed from the browser to the renderer with
DispatchInstallEvent. The ordering between RegisterForeignFetchScopes and the
callback of DispatchInstallEvent should be preserved.

+  Register(array<url.mojom.Url> sub_scopes, array<url.mojom.Origin> origins);

[shimazu, 2017/04/05 05:01:27]

Could you rename it to RegisterForeignFetchScopes?

+};