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

Side by Side Diff: content/renderer/service_worker/embedded_worker_instance_client_impl.cc

Issue 2307543002: ServiceWorker: Mojofication of EWInstance::StopWorker (Closed)
Patch Set: Incorporated with the review Created 4 years, 3 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 (c) 2016 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 #include "content/renderer/service_worker/embedded_worker_instance_client_impl.h " 5 #include "content/renderer/service_worker/embedded_worker_instance_client_impl.h "
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "content/child/scoped_child_process_reference.h" 10 #include "content/child/scoped_child_process_reference.h"
11 #include "content/common/service_worker/embedded_worker_messages.h" 11 #include "content/common/service_worker/embedded_worker_messages.h"
12 #include "content/public/common/content_client.h" 12 #include "content/public/common/content_client.h"
13 #include "content/renderer/service_worker/embedded_worker_devtools_agent.h" 13 #include "content/renderer/service_worker/embedded_worker_devtools_agent.h"
14 #include "content/renderer/service_worker/service_worker_context_client.h" 14 #include "content/renderer/service_worker/service_worker_context_client.h"
15 #include "third_party/WebKit/public/web/WebEmbeddedWorker.h" 15 #include "third_party/WebKit/public/web/WebEmbeddedWorker.h"
16 #include "third_party/WebKit/public/web/WebEmbeddedWorkerStartData.h" 16 #include "third_party/WebKit/public/web/WebEmbeddedWorkerStartData.h"
17 17
18 namespace content { 18 namespace content {
19 19
20 // static 20 // static
21 void EmbeddedWorkerInstanceClientImpl::Create( 21 void EmbeddedWorkerInstanceClientImpl::Create(
22 EmbeddedWorkerDispatcher* dispatcher, 22 EmbeddedWorkerDispatcher* dispatcher,
23 mojo::InterfaceRequest<mojom::EmbeddedWorkerInstanceClient> request) { 23 mojo::InterfaceRequest<mojom::EmbeddedWorkerInstanceClient> request) {
24 new EmbeddedWorkerInstanceClientImpl(dispatcher, std::move(request)); 24 new EmbeddedWorkerInstanceClientImpl(dispatcher, std::move(request));
25 } 25 }
26 26
27 void EmbeddedWorkerInstanceClientImpl::StopWorkerCompleted() {
28 DCHECK(embedded_worker_id_);
29 DCHECK(stop_callback_);
30 dispatcher_->UnregisterWorker(embedded_worker_id_.value());
31 embedded_worker_id_.reset();
32 stop_callback_.Run();
33 TRACE_EVENT0("ServiceWorker",
34 "EmbeddedWorkerInstanceClientImpl::StopWorkerCompleted");
35 stop_callback_.Reset();
36 }
37
27 void EmbeddedWorkerInstanceClientImpl::StartWorker( 38 void EmbeddedWorkerInstanceClientImpl::StartWorker(
28 const EmbeddedWorkerStartParams& params) { 39 const EmbeddedWorkerStartParams& params) {
29 TRACE_EVENT0("ServiceWorker", 40 TRACE_EVENT0("ServiceWorker",
30 "EmbeddedWorkerInstanceClientImpl::StartWorker"); 41 "EmbeddedWorkerInstanceClientImpl::StartWorker");
31 42
43 embedded_worker_id_ = params.embedded_worker_id;
44
32 std::unique_ptr<EmbeddedWorkerDispatcher::WorkerWrapper> wrapper = 45 std::unique_ptr<EmbeddedWorkerDispatcher::WorkerWrapper> wrapper =
33 dispatcher_->StartWorkerContext( 46 dispatcher_->StartWorkerContext(
34 params, 47 params, base::MakeUnique<ServiceWorkerContextClient>(
35 base::MakeUnique<ServiceWorkerContextClient>( 48 params.embedded_worker_id,
36 params.embedded_worker_id, params.service_worker_version_id, 49 params.service_worker_version_id, params.scope,
37 params.scope, params.script_url, 50 params.script_url, params.worker_devtools_agent_route_id,
38 params.worker_devtools_agent_route_id)); 51 std::move(temporal_self_)));
39 wrapper_ = wrapper.get(); 52 wrapper_ = wrapper.get();
40 dispatcher_->RegisterWorker(params.embedded_worker_id, std::move(wrapper)); 53 dispatcher_->RegisterWorker(params.embedded_worker_id, std::move(wrapper));
41 } 54 }
42 55
56 void EmbeddedWorkerInstanceClientImpl::StopWorker(
57 const StopWorkerCallback& callback) {
58 DCHECK(embedded_worker_id_);
59 DCHECK(!stop_callback_);
60 TRACE_EVENT0("ServiceWorker", "EmbeddedWorkerInstanceClientImpl::StopWorker");
61 stop_callback_ = std::move(callback);
62 dispatcher_->RecordStopWorkerTimer(embedded_worker_id_.value());
63 wrapper_->worker()->terminateWorkerContext();
64 }
65
43 EmbeddedWorkerInstanceClientImpl::EmbeddedWorkerInstanceClientImpl( 66 EmbeddedWorkerInstanceClientImpl::EmbeddedWorkerInstanceClientImpl(
44 EmbeddedWorkerDispatcher* dispatcher, 67 EmbeddedWorkerDispatcher* dispatcher,
45 mojo::InterfaceRequest<mojom::EmbeddedWorkerInstanceClient> request) 68 mojo::InterfaceRequest<mojom::EmbeddedWorkerInstanceClient> request)
46 : dispatcher_(dispatcher), binding_(this, std::move(request)) { 69 : dispatcher_(dispatcher),
70 binding_(this, std::move(request)),
71 temporal_self_(std::unique_ptr<EmbeddedWorkerInstanceClientImpl>(this)) {
47 binding_.set_connection_error_handler(base::Bind( 72 binding_.set_connection_error_handler(base::Bind(
48 &EmbeddedWorkerInstanceClientImpl::OnError, base::Unretained(this))); 73 &EmbeddedWorkerInstanceClientImpl::OnError, base::Unretained(this)));
49 } 74 }
50 75
51 EmbeddedWorkerInstanceClientImpl::~EmbeddedWorkerInstanceClientImpl() {} 76 EmbeddedWorkerInstanceClientImpl::~EmbeddedWorkerInstanceClientImpl() {}
52 77
53 void EmbeddedWorkerInstanceClientImpl::OnError() { 78 void EmbeddedWorkerInstanceClientImpl::OnError() {
54 // TODO(shimazu): Implement here 79 // Removes myself if it's owned by myself.
55 NOTIMPLEMENTED(); 80 temporal_self_.reset();
56 } 81 }
57 82
58 } // namespace content 83 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698