Commit e04f3055 authored by Rayan Kanso's avatar Rayan Kanso Committed by Commit Bot

[Background Sync] Remove the TODO about waiting for an installing worker

The new version of the spec removes the clause about waiting for
installing workers since that can easily cause deadlocks.

Bug: 542437
Change-Id: I4192787248e3d147ec001516f86aa17ea361752f
Reviewed-on: https://chromium-review.googlesource.com/c/1323717Reviewed-by: default avatarJosh Karlin <jkarlin@chromium.org>
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606418}
parent 1c1528b7
...@@ -25,8 +25,6 @@ SyncManager::SyncManager(ServiceWorkerRegistration* registration) ...@@ -25,8 +25,6 @@ SyncManager::SyncManager(ServiceWorkerRegistration* registration)
ScriptPromise SyncManager::registerFunction(ScriptState* script_state, ScriptPromise SyncManager::registerFunction(ScriptState* script_state,
const String& tag) { const String& tag) {
// TODO(jkarlin): Wait for the registration to become active instead of
// rejecting. See crbug.com/542437.
if (!registration_->active()) { if (!registration_->active()) {
return ScriptPromise::RejectWithDOMException( return ScriptPromise::RejectWithDOMException(
script_state, script_state,
......
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