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

RESOLVED FIXED in Firefox 43

Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: noemi, Assigned: nsm)

Tracking

Trunk
mozilla43
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 affected, firefox43 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

4 years ago
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+
Reporter

Comment 3

4 years ago
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/mozilla-central/rev/abec11a10c3d
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Reporter

Comment 6

4 years ago
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.