Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(59)

Side by Side Diff: content/common/service_worker/service_worker.mojom

Issue 2653493009: Add two interfaces for ServiceWorkerProviderContext/ProviderHost (Closed)
Patch Set: Rebased/Fixed unittests when BrowserSideNavigation is enabled Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 module content.mojom; 5 module content.mojom;
6 6
7 import "content/common/service_worker/service_worker_provider.mojom"; 7 import "content/common/service_worker/service_worker_provider.mojom";
8 8
9 // Per-process browser-side interface bound to ServiceWorkerDispatcherHost. 9 // Per-process browser-side interface bound to ServiceWorkerDispatcherHost.
falken 2017/05/18 04:13:09 content::ServiceWorkerDispatcherHost
shimazu 2017/05/19 08:31:07 Done.
10 // Each InterfacePtrs on the same render process will be bound to the same 10 // Each InterfacePtrs on the same render process will be bound to the same
falken 2017/05/18 04:13:10 All InterfacePtrs on the same renderer process are
shimazu 2017/05/19 08:31:07 Done.
11 // ServiceWorkerDispatcherHost. 11 // ServiceWorkerDispatcherHost.
falken 2017/05/18 04:13:09 Add sentence like: The renderer uses this interfac
shimazu 2017/05/19 08:31:07 Done.
12 interface ServiceWorkerDispatcherHost { 12 interface ServiceWorkerDispatcherHost {
13 // Creates a ServiceWorkerProviderHost on the browser process. |provider_info|
14 // have mojo endpoints to connect the provider host and the provider on the
falken 2017/05/18 04:13:10 has Mojo endpoints
shimazu 2017/05/19 08:31:06 Done.
15 // renderer together. The lifetime of ServiceWorkerProviderHost will be
16 // managed with the interface.
falken 2017/05/18 04:13:10 "managed with the interface" -> "tied to this mojo
shimazu 2017/05/19 08:31:07 Done.
13 OnProviderCreated(ServiceWorkerProviderHostInfo provider_info); 17 OnProviderCreated(ServiceWorkerProviderHostInfo provider_info);
14 OnProviderDestroyed(int32 provider_id);
15 18
16 // Informs the browser that a service worker is starting up. |provider_id| 19 // Informs the browser that a service worker is starting up. |provider_id|
17 // identifies the ServiceWorkerProviderHost hosting the service 20 // identifies the ServiceWorkerProviderHost hosting the service
18 // worker. |version_id| identifies the ServiceWorkerVersion and 21 // worker. |version_id| identifies the ServiceWorkerVersion and
19 // |embedded_worker_id| identifies the EmbeddedWorkerInstance. 22 // |embedded_worker_id| identifies the EmbeddedWorkerInstance.
20 OnSetHostedVersionId(int32 provider_id, 23 OnSetHostedVersionId(int32 provider_id,
21 int64 version_id, 24 int64 version_id,
22 int32 embedded_worker_id); 25 int32 embedded_worker_id);
23 }; 26 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698