Chromium Code Reviews
DescriptionBackgroundSync sync events need to be ExtendableEvents, content side
This is part of a series of patches to correctly fire the Sync event as
an ExtendableEvent, instead of a regular Event
[1/4] This patch, content side fix
[2/4] https://codereview.chromium.org/1148293011/, blink side fix
[3/4] https://codereview.chromium.org/1160253004/, content browser tests
[4/4] https://codereview.chromium.org/1146063005/, cleanup
Specific changes in this patch
- Sync event finished handling accepts the result of the waitUntil promise
- The worker status will be set correctly if the promise was rejected
- Overload of existing public/web interface provided for compatibility
with blink implementation
BUG=486890
Committed: https://crrev.com/b2bc492b3a8709bd3398cf4ea18ddc1909286d2d
Cr-Commit-Position: refs/heads/master@{#332787}
Patch Set 1 #Patch Set 2 : Avoid race condition in sync event dispatch #
Messages
Total messages: 11 (3 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||