[wpt-sync] Sync PR 13240 - WPT: service worker: Fix potential race in navigation-redirect.https.html.

RESOLVED FIXED in Firefox 64

Status

enhancement
P4
normal
RESOLVED FIXED
9 months ago
9 months ago

People

(Reporter: wptsync, Unassigned)

Tracking

unspecified
mozilla64
Points:
---

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: [wptsync downstream], )

Assignee

Description

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

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

Matt Falkenhagen <falken@chromium.org> wrote:
>  WPT: service worker: Fix potential race in navigation-redirect.https.html.
>  
>  Before this CL, the test makes a frame which posts a message to the test
>  window. But if the test fails, we might not consume the message with
>  `await` and move on the next test. It's possible the next test gets an
>  unexpected message from the previous test.
>  
>  I think a related issue caused the timeout after the Clients variant was
>  added to the test, since the message handler was created after the frame
>  was created and it was possible the message arrived after that.
>  
>  Fix things by explicitly asking the frame for the message when we're
>  ready for it.
>  
>  Bug: 876223, 778497
>  Change-Id: I82b8685df2977aeadea7475965ecccf69bd33111
>  Reviewed-on: https://chromium-review.googlesource.com/1249423
>  WPT-Export-Revision: 0f8b2fb0e8dac8723274ca721a3cac4dd6c469c2
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Assignee

Updated

9 months ago
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Assignee

Comment 6

9 months ago
Ran 1 tests and 37 subtests
OK     : 1
PASS   : 37

Comment 7

9 months ago
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bb2525c6a01b
[wpt PR 13240] - WPT: service worker: Fix potential race in navigation-redirect.https.html., a=testonly

Comment 8

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/bb2525c6a01b
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.