Commit 8a3c27d3 authored by Kenichi Ishibashi's avatar Kenichi Ishibashi Committed by Commit Bot

Update class comments of ServiceWorkerObjectHost

Some comments were outdated and mentioned classes which no longer
exist. Update these comments.

Bug: N/A
Change-Id: I2264e8a664d2a74f56080f8a4ccdc98d1fd0a5e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1880239
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Reviewed-by: default avatarMatt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#709345}
parent 21ccaee2
...@@ -28,10 +28,11 @@ namespace service_worker_object_host_unittest { ...@@ -28,10 +28,11 @@ namespace service_worker_object_host_unittest {
class ServiceWorkerObjectHostTest; class ServiceWorkerObjectHostTest;
} // namespace service_worker_object_host_unittest } // namespace service_worker_object_host_unittest
// Roughly corresponds to one WebServiceWorker object in the renderer process. // Roughly corresponds to one blink::ServiceWorker object in the renderer
// process.
// //
// The WebServiceWorker object in the renderer process maintains a reference to // The blink::ServiceWorker object in the renderer process maintains a
// |this| by owning an associated interface pointer to // reference to |this| by owning a Mojo remote to
// blink::mojom::ServiceWorkerObjectHost. When all Mojo connections bound with // blink::mojom::ServiceWorkerObjectHost. When all Mojo connections bound with
// |receivers_| are disconnected, |this| will be deleted. See also comments on // |receivers_| are disconnected, |this| will be deleted. See also comments on
// |receivers_|. // |receivers_|.
...@@ -111,12 +112,12 @@ class CONTENT_EXPORT ServiceWorkerObjectHost ...@@ -111,12 +112,12 @@ class CONTENT_EXPORT ServiceWorkerObjectHost
const url::Origin provider_origin_; const url::Origin provider_origin_;
scoped_refptr<ServiceWorkerVersion> version_; scoped_refptr<ServiceWorkerVersion> version_;
// Typically both |receivers_| and |remote_objects_| contain only one Mojo // Typically both |receivers_| and |remote_objects_| contain only one Mojo
// connection, corresponding to the content::WebServiceWorkerImpl in the // connection, corresponding to the blink::ServiceWorker in the renderer which
// renderer which corresponds to the ServiceWorker JavaScript object. However, // corresponds to the ServiceWorker JavaScript object. However, multiple Mojo
// multiple Mojo connections may exist while propagating multiple service // connections may exist while propagating multiple service worker object
// worker object infos to the renderer process, but only the first one that // infos to the renderer process, but only the first one that arrived there
// arrived there will be used to create the new content::WebServiceWorkerImpl // will be used to create the new blink::ServiceWorker instance and be bound
// instance and be bound to it. // to it.
mojo::AssociatedReceiverSet<blink::mojom::ServiceWorkerObjectHost> receivers_; mojo::AssociatedReceiverSet<blink::mojom::ServiceWorkerObjectHost> receivers_;
mojo::AssociatedRemoteSet<blink::mojom::ServiceWorkerObject> remote_objects_; mojo::AssociatedRemoteSet<blink::mojom::ServiceWorkerObject> remote_objects_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment