[ServiceWorker] Mojofy InstallEvent of Service Worker

content/common/service_worker/service_worker_event_dispatcher.mojom

 // Copyright 2016 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.
 
 module content.mojom;
 
 import "components/payments/content/payment_app.mojom";
 import "content/common/url_loader.mojom";
 import "mojo/common/string16.mojom";
 import "mojo/common/time.mojom";
 import "third_party/WebKit/public/platform/modules/background_sync/background_sync.mojom";
 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;
 
 [Native]
 struct PushEventPayload;
 
 [Native]
 struct ServiceWorkerResponse;
 
@@ skipping 21 matching lines @@
 struct ExtendableMessageEvent {
   mojo.common.mojom.String16 message;
   url.mojom.Origin source_origin;
   array<handle<message_pipe>> message_ports;
   ExtendableMessageEventSource source;
 };
 
 // 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);
   DispatchBackgroundFetchAbortEvent(string tag)
       => (blink.mojom.ServiceWorkerEventStatus status,
           mojo.common.mojom.Time dispatch_event_time);
   DispatchBackgroundFetchClickEvent(string tag, BackgroundFetchState state)
       => (blink.mojom.ServiceWorkerEventStatus status,
           mojo.common.mojom.Time dispatch_event_time);
   DispatchBackgroundFetchFailEvent(string tag,
@@ skipping 37 matching lines @@
   DispatchExtendableMessageEvent(ExtendableMessageEvent event)
       => (blink.mojom.ServiceWorkerEventStatus status,
           mojo.common.mojom.Time dispatch_event_time);
 
   // Pings the service worker to check if it is responsive. If the callback is
   // not called within a certain period of time, the browser will terminate the
   // worker. Unlike the other functions in this interface, Ping() does not
   // 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?

+};