1691764 - [wpt-sync] Sync PR 27549 - Revert "Update registration-schedule-job.https.html to reflect the spec"

Status: RESOLVED FIXED

(Core :: DOM: Service Workers, task, P4)

Description

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

Sync web-platform-tests PR 27549 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/27549
Details from upstream follow.

b'Christian Dullweber <[e-mail]>' wrote:

Revert "Update registration-schedule-job.https.html to reflect the spec"

This reverts commit cf04532ef9c4b904f22d1307bd924b90bbbe4ddb.

Reason for revert: The test started failing on mac after CL landed:
https://ci.chromium.org/ui/p/chromium/builders/ci/Mac10.12%20Tests/41681/overview

Original change's description:

Update registration-schedule-job.https.html to reflect the spec

Previously, the test expectation didn't comply with the spec because it
expected there's an installing worker even if register() actually didn't
install a new worker.
This CL is to update the expectation to wait for each register() call
resolved one by one. This should work because each job must not be
coalesced so the states should be updated correctly when those promises
are resolved.

Bug: 979593
Change-Id: Ib3874e7e15ebb0dfa5608cc2ff4d3484cffd72b2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2679094
Commit-Queue: Makoto Shimazu \<[e-mail]>
Reviewed-by: Asami Doi \<[e-mail]>
Cr-Commit-Position: refs/heads/master@{#852157}

TBR=[e-mail],[e-mail],[e-mail]

Change-Id: Icb7c609f6b8a44b55ae338255058679d7fa55624
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 979593
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2684257
Reviewed-by: Christian Dullweber \<[e-mail]>
Commit-Queue: Christian Dullweber \<[e-mail]>
Cr-Commit-Position: refs/heads/master@{#852210}

Comment 1

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=335181accfa109b418cfe0c1e6d6166c47d968bb

Comment 2

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

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 2 subtests

Status Summary

Firefox

OK : 1
FAIL: 2

Chrome

OK : 1
FAIL: 2

Safari

OK : 1
FAIL: 2

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/service-workers/service-worker/registration-schedule-job.https.html
different scriptURL and updateViaCache: FAIL (Chrome: FAIL, Safari: FAIL)
different type: FAIL (Chrome: FAIL, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c0d451a66d53
[wpt PR 27549] - Revert "Update registration-schedule-job.https.html to reflect the spec", a=testonly

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c498c025859e
[wpt PR 27549] - Revert "Update registration-schedule-job.https.html to reflect the spec", a=testonly

Comment 5

[Cristian Brindusan [:cbrindusan]]

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