Closed Bug 1189653 Opened 5 years ago Closed 5 years ago
"Harness status: Timeout" when running "fetch-request-css-base-url
.https .html" test
40 bytes, text/x-review-board-request
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
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
Attachment #8659608 - Flags: review?(bkelly)
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
Attachment #8659608 - Flags: review?(bkelly) → review+
Patch provided by Nikhil so assigning this to him. Please feel free to change it as needed. Thanks!
Assignee: nobody → nsm.nikhil
Status: NEW → ASSIGNED
https://hg.mozilla.org/integration/mozilla-inbound/rev/abec11a10c3d2661d28215472a6b004888c130ba Bug 1189653 - Wait for frame to be loaded and test to finish. r=bkelly
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.
You need to log in before you can comment on or make changes to this bug.