1189653 - "Harness status: Timeout" when running "fetch-request-css-base-url.https.html" test

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect)

Description

[Noemí Freire (:noemi)]

Test run such as |./mach web-platform-tests _mozilla/service-workers/service-worker/fetch-request-css-base-url.https.html|

Actual Result:
Harness status: Timeout
Found 1 tests
1 Timeout
* CSS's base URL must be the request URL even when fetched from other URL -> timed out
Traces: https://pastebin.mozilla.org/8838594

Comment 1

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Attachment: MozReview Request: Bug 1189653 - Wait for frame to be loaded and test to finish. r?bkelly

Bug 1189653 - Wait for frame to be loaded and test to finish. r?bkelly

This test seems to work on central but fails due to a timing issue where the
channel may receive the message before the with_iframe() call resolves the next
promise to set the global variable `frame`.  This patch tweaks the test to
resolve a promise instead when the test is done and then wait on the
with_iframe() promise and the test promise.

Update web-platform-tests expected data

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8659608 [details]
MozReview Request: Bug 1189653 - Wait for frame to be loaded and test to finish. r?bkelly

https://reviewboard.mozilla.org/r/18985/#review16863

Comment 3

[Noemí Freire (:noemi)]

Patch provided by Nikhil so assigning this to him. Please feel free to change it as needed. Thanks!

Comment 4

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/abec11a10c3d2661d28215472a6b004888c130ba
Bug 1189653 - Wait for frame to be loaded and test to finish. r=bkelly

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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

Comment 6

[Noemí Freire (:noemi)]

Hi,

just checked in  m-c (c69e31de9aec revision) and the test successfully runs. Thanks for fixing it!.

Summary

Harness status: OK

Found 1 tests
1 Pass
Details
Result	Test Name
Pass	CSS's base URL must be the request URL even when fetched from other URL.