New issue
Advanced search Search tips

Issue 649351 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 646783
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocked on:
issue 579137



Sign in to add a comment

Limit the maximum execution time of the `push` event

Project Member Reported by peter@chromium.org, Sep 22 2016

Issue description

The `push` event in Service Workers currently doesn't have an explicit timeout assigned to it. This has led to some developers passing a never resolving promise to the event's waitUntil() method, which is undesirable.

The ServiceWorker.PushEvent.Time UMA shows that the 99th percentile is 23 seconds, 99.9th a minute, and 99.99th three minutes. We're hoping that this will decrease further as more developers adopt payloads.

We can match Background Sync's timeout of three minutes, or be more aggressive and go for one minute. In either case, we should fix  Issue 579137  to make sure that *a notification*, with or without images, can be shown within the timeout.
 

Comment 1 by falken@chromium.org, Sep 22 2016

Same as  Issue 646783 ?

Comment 2 by peter@chromium.org, Sep 22 2016

Mergedinto: 646783
Status: Duplicate (was: Assigned)
Ah, John filed it :) We couldn't find it.

Sign in to add a comment