Closed Bug 1161036 Opened 4 years ago Closed 4 years ago

test_interfaces.html and test_serviceworker_interfaces.html are going to permafail when Gecko 40 merges to Beta

Categories

(Core :: DOM: Core & HTML, defect, major)

defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla40
Tracking Status
firefox40 --- verified

People

(Reporter: RyanVM, Assigned: bzbarsky)

References

Details

Attachments

(2 files)

https://treeherder.mozilla.org/logviewer.html#?job_id=7172244&repo=try

16:44:49 INFO - 158 INFO TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | If this is failing: DANGER, are you sure you want to expose the new interface FontFace to all webpages as a property on the window (XBL: false)? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS

16:44:50 INFO - 298 INFO TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | If this is failing: DANGER, are you sure you want to expose the new interface FontFaceSet to all webpages as a property on the window (XBL: false)? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS

16:44:50 INFO - 480 INFO TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | If this is failing: DANGER, are you sure you want to expose the new interface CSSFontFaceLoadEvent to all webpages as a property on the window (XBL: false)? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS

16:44:51 INFO - 772 INFO TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | If this is failing: DANGER, are you sure you want to expose the new interface FontFace to all webpages as a property on the window (XBL: true)? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS

16:44:52 INFO - 929 INFO TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | If this is failing: DANGER, are you sure you want to expose the new interface FontFaceSet to all webpages as a property on the window (XBL: true)? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS

16:44:53 INFO - 1129 INFO TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | If this is failing: DANGER, are you sure you want to expose the new interface CSSFontFaceLoadEvent to all webpages as a property on the window (XBL: true)? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS

16:47:32 INFO - 1791 INFO TEST-UNEXPECTED-FAIL | dom/workers/test/serviceworkers/test_serviceworker_interfaces.html | false: If this is failing: DANGER, are you sure you want to expose the new interface Cache to all webpages as a property on the service worker? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS

16:47:32 INFO - 1792 INFO TEST-UNEXPECTED-FAIL | dom/workers/test/serviceworkers/test_serviceworker_interfaces.html | false: If this is failing: DANGER, are you sure you want to expose the new interface CacheStorage to all webpages as a property on the service worker? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) - expected PASS
Flags: needinfo?(bzbarsky)
Hrm.  Looks like I didn't notice that the test harness sets this pref unconditionally.  Need to do a try run to see whether all the tests that use this actually set the pref.  :(
Flags: needinfo?(bzbarsky)
"This pref" == layout.css.font-loading-api.enabled.  But also, they set "dom.caches.enabled"...
Except not.  Looking into why the serviceworker bit fails.
So test_serviceworker_interfaces.html explicitly sets the cache pref.  Patch coming up per discussion with bkelly for that part too.
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #8601234 - Flags: review?(cam) → review+
Attachment #8601235 - Flags: review?(bkelly) → review+
https://hg.mozilla.org/mozilla-central/rev/346bc2006741
https://hg.mozilla.org/mozilla-central/rev/4e89ee947dd5
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Status: RESOLVED → VERIFIED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.