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

Unified Diff: content/browser/service_worker/embedded_worker_instance.cc

Issue 2787883003: [ServiceWorker] Add EmbeddedWorkerInstanceHost Interface. (Closed)
Patch Set: Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/service_worker/embedded_worker_instance.cc
diff --git a/content/browser/service_worker/embedded_worker_instance.cc b/content/browser/service_worker/embedded_worker_instance.cc
index 727f2d548847080b097b6c2dbad2333621aace02..6feee0a340c81c8cafe03ac86fc79bbfa2e36398 100644
--- a/content/browser/service_worker/embedded_worker_instance.cc
+++ b/content/browser/service_worker/embedded_worker_instance.cc
@@ -12,6 +12,7 @@
#include "base/metrics/histogram_macros.h"
#include "base/threading/non_thread_safe.h"
#include "base/trace_event/trace_event.h"
+#include "content/browser/bad_message.h"
#include "content/browser/devtools/service_worker_devtools_manager.h"
#include "content/browser/service_worker/embedded_worker_registry.h"
#include "content/browser/service_worker/embedded_worker_status.h"
@@ -487,9 +488,8 @@ bool EmbeddedWorkerInstance::Stop() {
OnDetached();
return false;
}
- client_->StopWorker(base::Bind(&EmbeddedWorkerRegistry::OnWorkerStopped,
- base::Unretained(registry_.get()),
- process_id(), embedded_worker_id()));
+ client_->StopWorker(
+ base::Bind(&EmbeddedWorkerInstance::OnStopped, base::Unretained(this)));
} else {
ServiceWorkerStatusCode status =
registry_->StopWorker(process_id(), embedded_worker_id_);
@@ -556,6 +556,7 @@ EmbeddedWorkerInstance::EmbeddedWorkerInstance(
starting_phase_(NOT_STARTING),
restart_count_(0),
thread_id_(kInvalidEmbeddedWorkerThreadId),
+ instance_host_binding_(this),
devtools_attached_(false),
network_accessed_for_script_(false),
weak_factory_(this) {}
@@ -595,8 +596,13 @@ ServiceWorkerStatusCode EmbeddedWorkerInstance::SendStartWorker(
std::unique_ptr<EmbeddedWorkerStartParams> params) {
if (!context_)
return SERVICE_WORKER_ERROR_ABORT;
+
+ mojom::EmbeddedWorkerInstanceHostAssociatedPtrInfo host_ptr_info;
+ instance_host_binding_.Bind(&host_ptr_info);
+
DCHECK(pending_dispatcher_request_.is_pending());
- client_->StartWorker(*params, std::move(pending_dispatcher_request_));
+ client_->StartWorker(*params, std::move(pending_dispatcher_request_),
+ std::move(host_ptr_info));
registry_->BindWorkerToProcess(process_id(), embedded_worker_id());
TRACE_EVENT_ASYNC_STEP_PAST0("ServiceWorker", "EmbeddedWorkerInstance::Start",
this, "SendStartWorker");
@@ -619,6 +625,7 @@ void EmbeddedWorkerInstance::OnStartWorkerMessageSent() {
}
void EmbeddedWorkerInstance::OnReadyForInspection() {
+ TRACE_EVENT0("ServiceWorker", "EmbeddedWorkerInstance::OnReadyForInspection");
if (devtools_proxy_)
devtools_proxy_->NotifyWorkerReadyForInspection();
}
@@ -634,6 +641,8 @@ void EmbeddedWorkerInstance::OnScriptReadFinished() {
void EmbeddedWorkerInstance::OnScriptLoaded() {
using LoadSource = ServiceWorkerMetrics::LoadSource;
+ TRACE_EVENT0("ServiceWorker", "EmbeddedWorkerInstance::OnScriptLoaded");
+
if (!inflight_start_task_)
return;
LoadSource source;
@@ -683,7 +692,21 @@ void EmbeddedWorkerInstance::OnWorkerVersionDoomed() {
devtools_proxy_->NotifyWorkerVersionDoomed();
}
-void EmbeddedWorkerInstance::OnThreadStarted(int thread_id) {
+void EmbeddedWorkerInstance::OnThreadStarted(int thread_id, int provider_id) {
+ TRACE_EVENT0("ServiceWorker", "EmbeddedWorkerInstance::OnThreadStarted");
+ if (!context_)
+ return;
+
+ ServiceWorkerProviderHost* provider_host =
+ context_->GetProviderHost(process_id(), provider_id);
+ if (!provider_host) {
+ bad_message::ReceivedBadMessage(
+ process_id(), bad_message::SWDH_WORKER_SCRIPT_LOAD_NO_HOST);
+ return;
+ }
+
+ provider_host->SetReadyToSendMessagesToWorker(thread_id);
+
if (!inflight_start_task_)
return;
TRACE_EVENT_ASYNC_STEP_PAST0("ServiceWorker", "EmbeddedWorkerInstance::Start",
@@ -702,6 +725,7 @@ void EmbeddedWorkerInstance::OnThreadStarted(int thread_id) {
}
void EmbeddedWorkerInstance::OnScriptLoadFailed() {
+ TRACE_EVENT0("ServiceWorker", "EmbeddedWorkerInstance::OnScriptLoadFailed");
if (!inflight_start_task_)
return;
TRACE_EVENT_ASYNC_STEP_PAST0("ServiceWorker", "EmbeddedWorkerInstance::Start",
@@ -712,6 +736,7 @@ void EmbeddedWorkerInstance::OnScriptLoadFailed() {
}
void EmbeddedWorkerInstance::OnScriptEvaluated(bool success) {
+ TRACE_EVENT0("ServiceWorker", "EmbeddedWorkerInstance::OnScriptEvaluated");
if (!inflight_start_task_)
return;
DCHECK_EQ(EmbeddedWorkerStatus::STARTING, status_);
@@ -736,6 +761,9 @@ void EmbeddedWorkerInstance::OnScriptEvaluated(bool success) {
}
void EmbeddedWorkerInstance::OnStarted() {
+ TRACE_EVENT0("ServiceWorker", "EmbeddedWorkerInstance::OnStarted");
+ if (!registry_->OnWorkerStarted(process_id(), embedded_worker_id_))
leonhsl(Using Gerrit) 2017/04/04 07:14:04 Need to do some necessary work in registry.
+ return;
// Stop is requested before OnStarted is sent back from the worker.
if (status_ == EmbeddedWorkerStatus::STOPPING)
return;
@@ -747,6 +775,8 @@ void EmbeddedWorkerInstance::OnStarted() {
}
void EmbeddedWorkerInstance::OnStopped() {
+ registry_->OnWorkerStoppedd(process_id(), embedded_worker_id_);
+
EmbeddedWorkerStatus old_status = status_;
ReleaseProcess();
for (auto& observer : listener_list_)

Powered by Google App Engine
This is Rietveld 408576698