Closed
Bug 1161036
Opened 9 years ago
Closed 9 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)
Core
DOM: Core & HTML
Tracking
()
VERIFIED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | verified |
People
(Reporter: RyanVM, Assigned: bzbarsky)
References
Details
Attachments
(2 files)
7.13 KB,
patch
|
heycam
:
review+
|
Details | Diff | Splinter Review |
3.36 KB,
patch
|
bkelly
:
review+
|
Details | Diff | Splinter Review |
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)
Assignee | ||
Comment 1•9 years ago
|
||
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)
Assignee | ||
Comment 2•9 years ago
|
||
"This pref" == layout.css.font-loading-api.enabled. But also, they set "dom.caches.enabled"...
Assignee | ||
Comment 3•9 years ago
|
||
Except not. Looking into why the serviceworker bit fails.
Assignee | ||
Comment 4•9 years ago
|
||
So test_serviceworker_interfaces.html explicitly sets the cache pref. Patch coming up per discussion with bkelly for that part too.
Assignee | ||
Comment 5•9 years ago
|
||
Attachment #8601234 -
Flags: review?(cam)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Assignee | ||
Comment 6•9 years ago
|
||
Attachment #8601235 -
Flags: review?(bkelly)
Updated•9 years ago
|
Attachment #8601234 -
Flags: review?(cam) → review+
Updated•9 years ago
|
Attachment #8601235 -
Flags: review?(bkelly) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/346bc2006741 https://hg.mozilla.org/integration/mozilla-inbound/rev/4e89ee947dd5
Comment 8•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/346bc2006741 https://hg.mozilla.org/mozilla-central/rev/4e89ee947dd5
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Reporter | ||
Updated•9 years ago
|
Status: RESOLVED → VERIFIED
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•