Closed Bug 1528032 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 15377 - WPT: service worker: Loosen postmessage-to-client-message-queue.https.html.

Categories

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

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

Tracking

(firefox67 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla67
Tracking Flags:
Tracking Status
firefox67 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Matt Falkenhagen <falken@chromium.org> wrote:

WPT: service worker: Loosen postmessage-to-client-message-queue.https.html.

Chrome flakily failed the test because it doesn't guarantee the order
between message and fetch event. That might violate the spec, but the
test doesn't need to test this. It's meant to test that the client
message queue isn't enabled until a certain time. We can test
that without requiring message/fetch ordering.

Loosen the test to just check that the messages arrive after
|earliest_dispatch|.

Bug: 926670
Change-Id: Ic291b25443358412936b4a5eeb24ee6f8efa6b71
Reviewed-on: https://chromium-review.googlesource.com/1469842
WPT-Export-Revision: 78b71d3c57d8f5ed54835ce9d3e1b9a503f5c58a

Ran 1 tests and 5 subtests
OK     : 1
PASS   : 5
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d3e10f10f4c2
[wpt PR 15377] - WPT: service worker: Loosen postmessage-to-client-message-queue.https.html., a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b172a1750aec
[wpt PR 15377] - WPT: service worker: Loosen postmessage-to-client-message-queue.https.html., a=testonly

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

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox67: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.