Closed Bug 1607136 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 21046 - Sort the registrations returned by getRegistrations() by registration ID.

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox74 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla74
Tracking Flags:
Tracking Status
firefox74 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Wei Wang <wei4.wang@intel.com> wrote:

Sort the registrations returned by getRegistrations() by registration ID.

This patch makes the order of the registrations returned by
getRegistrations() be sorted by registration ID, which fixes the flaky
test of getregistrations.https.html.

Bug:925740
Change-Id: Iaa5716eec886232df61bd23487fbae5c63413e9c
Reviewed-on: https://chromium-review.googlesource.com/1984900
WPT-Export-Revision: 617387a6b025913d8ca805a1f79de716db842947

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

CI Results

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

Total 42 tests

Status Summary

Firefox

OK : 1
PASS: 5[GitHub] 46[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt]
FAIL: 1

Chrome

OK : 1
PASS: 6

Safari

OK : 1
PASS: 6

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/service-workers/service-worker/getregistrations.https.html
Register multiple times then getRegistrations: FAIL

New Tests That Don't Pass

/service-workers/service-worker/getregistrations.https.html
Register multiple times then getRegistrations: FAIL (Chrome: PASS, Safari: PASS)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3cdd21c5e484
[wpt PR 21046] - Sort the registrations returned by getRegistrations() by registration ID., a=testonly
https://hg.mozilla.org/integration/autoland/rev/0231963a7be9
[wpt PR 21046] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/3cdd21c5e484
https://hg.mozilla.org/mozilla-central/rev/0231963a7be9

Status: REOPENED → RESOLVED
Closed: 4 years ago → 4 years ago
status-firefox74: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
You need to log in before you can comment on or make changes to this bug.