1458805 - Intermittent /fetch/api/abort/serviceworker-intercepted.https.html | response.blob() rejects if already aborted - promise_test: Unhandled rejection with value: object "Error: wait_for_state must be passed a ServiceWorker"

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: archaeopteryx [at] coole-files.de

https://treeherder.mozilla.org/logviewer.html#?job_id=176593027&repo=try

https://queue.taskcluster.net/v1/task/I-xw2rvzSZe8aJmPw4GjEA/runs/0/artifacts/public/logs/live_backing.log

[task 2018-05-02T16:55:28.485Z] 16:55:28     INFO - TEST-START | /fetch/api/abort/serviceworker-intercepted.https.html
[task 2018-05-02T16:55:28.487Z] 16:55:28     INFO - Clearing pref dom.streams.enabled
[task 2018-05-02T16:55:28.488Z] 16:55:28     INFO - Clearing pref javascript.options.streams
[task 2018-05-02T16:55:28.505Z] 16:55:28     INFO - Setting pref dom.streams.enabled (true)
[task 2018-05-02T16:55:28.513Z] 16:55:28     INFO - Setting pref javascript.options.streams (true)
[task 2018-05-02T16:55:29.278Z] 16:55:29     INFO - 
[task 2018-05-02T16:55:29.278Z] 16:55:29     INFO - TEST-PASS | /fetch/api/abort/serviceworker-intercepted.https.html | Already aborted request does not land in service worker 
[task 2018-05-02T16:55:29.280Z] 16:55:29     INFO - TEST-PASS | /fetch/api/abort/serviceworker-intercepted.https.html | response.arrayBuffer() rejects if already aborted 
[task 2018-05-02T16:55:29.280Z] 16:55:29     INFO - TEST-UNEXPECTED-FAIL | /fetch/api/abort/serviceworker-intercepted.https.html | response.blob() rejects if already aborted - promise_test: Unhandled rejection with value: object "Error: wait_for_state must be passed a ServiceWorker"
[task 2018-05-02T16:55:29.280Z] 16:55:29     INFO - ....
[task 2018-05-02T16:55:29.280Z] 16:55:29     INFO - TEST-OK | /fetch/api/abort/serviceworker-intercepted.https.html | took 799ms

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

This is due to a small bug in the test.

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Attachment: Make abort/serviceworker-intercepted.https.html WPT cases use unique scopes. r=baku

This makes the serviceworker-intercepted.https.html WPT test use separate scopes for each case.  This avoids a small race between unregistering the service worker in the previous case and registering the service worker in the next case.  The controlled frame is removed, but pending events can also keep the service worker registration alive for short periods of time.

Comment 3

[Ben Kelly [:bkelly, not reviewing]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=59aa98074478ec9478d2d8dc1acf1108fd7c4fb6

Comment 4

[Ben Kelly [:bkelly, not reviewing]]

The test also previously left a service worker registered since it did not call cleanup() after the last case.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f127f18d19a9
Make abort/serviceworker-intercepted.https.html WPT cases use unique scopes. r=baku

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/w3c/web-platform-tests/pull/10825 for changes under testing/web-platform/tests

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 8

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/f127f18d19a9

Comment 9

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged

Comment 10

[Intermittent Failures Robot]

1 failures in 551 pushes (0.002 failures/push) were associated with this bug in the last 7 days.    

Repository breakdown:
* try: 1

Platform breakdown:
* linux32-nightly: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1458805&startday=2018-04-30&endday=2018-05-06&tree=trunk