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

Unified Diff: content/renderer/service_worker/embedded_worker_context_client.cc

Issue 958523002: ServiceWorker: Use scheduler's default task queue for posting tasks on main thread (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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/renderer/service_worker/embedded_worker_context_client.cc
diff --git a/content/renderer/service_worker/embedded_worker_context_client.cc b/content/renderer/service_worker/embedded_worker_context_client.cc
index 2173cedeb59ad6dd5433434a5c31fe09509b1f12..b84340b5bcb8f166fcea25cad1f18bdecbfcd139 100644
--- a/content/renderer/service_worker/embedded_worker_context_client.cc
+++ b/content/renderer/service_worker/embedded_worker_context_client.cc
@@ -106,7 +106,7 @@ EmbeddedWorkerContextClient::EmbeddedWorkerContextClient(
script_url_(script_url),
worker_devtools_agent_route_id_(worker_devtools_agent_route_id),
sender_(ChildThreadImpl::current()->thread_safe_sender()),
- main_thread_proxy_(base::MessageLoopProxy::current()),
+ main_thread_task_runner_(RenderThreadImpl::current()->GetTaskRunner()),
weak_factory_(this) {
TRACE_EVENT_ASYNC_BEGIN0("ServiceWorker",
"EmbeddedWorkerContextClient::StartingWorkerContext",
@@ -180,7 +180,7 @@ void EmbeddedWorkerContextClient::workerReadyForInspection() {
}
void EmbeddedWorkerContextClient::workerContextFailedToStart() {
- DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
+ DCHECK(main_thread_task_runner_->RunsTasksOnCurrentThread());
DCHECK(!script_context_);
Send(new EmbeddedWorkerHostMsg_WorkerScriptLoadFailed(embedded_worker_id_));
@@ -243,7 +243,7 @@ void EmbeddedWorkerContextClient::workerContextDestroyed() {
// Now we should be able to free the WebEmbeddedWorker container on the
// main thread.
- main_thread_proxy_->PostTask(
+ main_thread_task_runner_->PostTask(
FROM_HERE,
base::Bind(&CallWorkerContextDestroyedOnMainThread,
embedded_worker_id_));
@@ -355,7 +355,7 @@ void EmbeddedWorkerContextClient::didHandleCrossOriginConnectEvent(
blink::WebServiceWorkerNetworkProvider*
EmbeddedWorkerContextClient::createServiceWorkerNetworkProvider(
blink::WebDataSource* data_source) {
- DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
+ DCHECK(main_thread_task_runner_->RunsTasksOnCurrentThread());
// Create a content::ServiceWorkerNetworkProvider for this data source so
// we can observe its requests.
@@ -380,7 +380,7 @@ EmbeddedWorkerContextClient::createServiceWorkerNetworkProvider(
blink::WebServiceWorkerProvider*
EmbeddedWorkerContextClient::createServiceWorkerProvider() {
- DCHECK(main_thread_proxy_->RunsTasksOnCurrentThread());
+ DCHECK(main_thread_task_runner_->RunsTasksOnCurrentThread());
DCHECK(provider_context_);
// Blink is responsible for deleting the returned object.

Powered by Google App Engine
This is Rietveld 408576698