New issue
Advanced search Search tips

Issue 621440 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: 2017-06-01
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Allow waitUntil to be called multiple times asynchronously

Project Member Reported by shimazu@chromium.org, Jun 20 2016

Issue description

Components: Blink>ServiceWorker
In the meantime, here's a polyfill https://github.com/jakearchibald/async-waituntil-polyfill

Comment 3 by falken@chromium.org, Jan 19 2017

Labels: -Pri-3 M-58 Pri-2
From https://github.com/w3c/ServiceWorker/issues/771:
"FYI, @catalinb has implemented this in gecko. It will ship in FF53 around April 2017."
NextAction: 2017-02-06
I'll get back here later to implement this in M58.

Comment 5 by horo@chromium.org, Feb 10 2017

ping?

Comment 6 by falken@chromium.org, Feb 21 2017

NextAction: 2017-04-01
Reset NextAction.

Comment 7 by falken@chromium.org, Apr 19 2017

Labels: -Pri-2 -M-58 Pri-3
NextAction: 2017-06-01
(blink worker triage) Reset NextAction.

Comment 8 by leon....@intel.com, May 15 2017

Cc: shimazu@chromium.org
Owner: leon....@intel.com
Labels: -Pri-3 Hotlist-Google M-61 Pri-2
We've heard from a few partners that it would be nice to not have to include a polyfill for this => Bumping the priority to 2 (desirable).

Leon, is this something you can work on soon?
Thanks a lot in advance!
Status: Started (was: Assigned)
Leon is creating a patch already: https://crrev.com/2877543003 .
Project Member

Comment 11 by bugdroid1@chromium.org, May 25 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eaf348dc9864d8073fe01472a91c33f21e3ec4b6

commit eaf348dc9864d8073fe01472a91c33f21e3ec4b6
Author: leon.han <leon.han@intel.com>
Date: Thu May 25 11:43:38 2017

[ServiceWorker] Allow waitUntil to be called multiple times asynchronously

Spec:
https://w3c.github.io/ServiceWorker/#dom-extendableevent-waituntil

BUG= 621440 
TEST=blink_tests
external/wpt/service-workers/service-worker/extendable-event-async-waituntil.https.html

Review-Url: https://codereview.chromium.org/2877543003
Cr-Commit-Position: refs/heads/master@{#474630}

[delete] https://crrev.com/929808a8f063d31a01b883ab8c81c21b838ad61b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/extendable-event-async-waituntil.https-expected.txt
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/extendable-event-async-waituntil.https.html
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/resources/extendable-event-async-waituntil.js
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/sw-test-helpers.js
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/windowclient-navigate-worker.js
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/Source/modules/serviceworkers/ExtendableEvent.cpp
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/Source/modules/serviceworkers/RespondWithObserver.cpp
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/Source/modules/serviceworkers/RespondWithObserver.h
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/Source/modules/serviceworkers/WaitUntilObserver.cpp
[modify] https://crrev.com/eaf348dc9864d8073fe01472a91c33f21e3ec4b6/third_party/WebKit/Source/modules/serviceworkers/WaitUntilObserver.h

The NextAction date has arrived: 2017-06-01
Status: Fixed (was: Started)
It's fixed by c#11. 
Labels: -M-61 M-60

Sign in to add a comment