Chromium Code Reviews| Index: content/child/service_worker/web_service_worker_impl.h |
| diff --git a/content/child/service_worker/web_service_worker_impl.h b/content/child/service_worker/web_service_worker_impl.h |
| index cf9728e5cbfffc7fb0f93bb1d5fa5b83fcae4556..205fc88139f2a7ea6f2886de4f2a8a1762bfffb9 100644 |
| --- a/content/child/service_worker/web_service_worker_impl.h |
| +++ b/content/child/service_worker/web_service_worker_impl.h |
| @@ -20,14 +20,25 @@ class WebServiceWorkerProxy; |
| namespace content { |
| -class ThreadSafeSender; |
| +class ServiceWorkerHandleReference; |
| struct ServiceWorkerObjectInfo; |
| +class ThreadSafeSender; |
| +// Each instance corresponds to onw ServiceWorker object in JS context, and |
|
michaeln
2014/05/05 23:18:03
own --> one?
kinuko
2014/05/06 00:39:37
Done.
|
| +// is held by ServiceWorker object in blink's c++ layer, e.g. created one |
| +// per navigator.serviceWorker.current or per successful |
| +// navigator.serviceWorker.register call. |
| +// |
| +// Each instance holds one ServiceWorkerHandleReference so that |
| +// corresponding ServiceWorkerHandle doesn't go away in the browser process |
| +// while the ServiceWorker object is alive. |
| class WebServiceWorkerImpl |
| : NON_EXPORTED_BASE(public blink::WebServiceWorker) { |
| public: |
| WebServiceWorkerImpl(const ServiceWorkerObjectInfo& info, |
| ThreadSafeSender* thread_safe_sender); |
| + WebServiceWorkerImpl(scoped_ptr<ServiceWorkerHandleReference> handle_ref, |
| + ThreadSafeSender* thread_safe_sender); |
| virtual ~WebServiceWorkerImpl(); |
| // Notifies that the service worker's state changed. This function may queue |
| @@ -47,9 +58,7 @@ class WebServiceWorkerImpl |
| // Commits the new state internally and notifies the proxy of the change. |
| void ChangeState(blink::WebServiceWorkerState new_state); |
| - const int handle_id_; |
| - const GURL scope_; |
| - const GURL url_; |
| + scoped_ptr<ServiceWorkerHandleReference> handle_ref_; |
| blink::WebServiceWorkerState state_; |
| scoped_refptr<ThreadSafeSender> thread_safe_sender_; |
| blink::WebServiceWorkerProxy* proxy_; |