Remove dom/imptests/ as redundant
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Tracking
()
People
(Reporter: hsivonen, Assigned: Ms2ger)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Reporter | ||
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
Comment 3•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Comment 4•8 years ago
|
||
Comment 5•8 years ago
|
||
Comment 6•8 years ago
|
||
Comment 7•8 years ago
|
||
Comment 8•8 years ago
|
||
Updated•7 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 11•5 years ago
|
||
This leaves the testharness files, because they are used in mochitest-chrome
tests in dom/animation/test.
Comment 12•5 years ago
|
||
Comment 17•5 years ago
|
||
Backed out changeset 73cbc0f61053 (Bug 1194860) for causing lint failure in /builds/worker/checkouts/gecko/dom/imptests/testharness.js CLOSED TREE
https://treeherder.mozilla.org/logviewer.html#?job_id=271133715&repo=autoland
Comment 18•5 years ago
|
||
Comment 21•5 years ago
|
||
Comment 22•5 years ago
|
||
Backed out changeset 8fb923925923 (Bug 1194860) for causing perma mda3 failures in dom/media/webaudio/test/test_convolverNodeChannelInterpretationChanges.html CLOSED TREE
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271157638&repo=autoland&lineNumber=15162
Comment 23•5 years ago
|
||
Comment 24•5 years ago
|
||
These tests all have equivalents in the WebGL CTS (dom/canvas/test/webgl-conf), fwiw!
Comment 25•5 years ago
|
||
There are more failures (mostly timeouts) caused by the push that was backed out:
- test_document-timeline.html: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158886&repo=autoland&lineNumber=4178
- test_bug1259588.html, test_Object.prototype_props.html, test_ImageData_ctor.html, test_TextDecoder.html, test_error_events.html:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158943&repo=autoland&lineNumber=1819 - test_bug1293174_implicit_pointer_capture_for_touch_1.html, test_bug1293174_implicit_pointer_capture_for_touch_2.html, test_bug1303704.html, test_bug1315862.html, test_form_named_getter_dynamic.html, test_bug780993.html:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158951&repo=autoland&lineNumber=1596 - test_performance_server_timing_plain_http.html, test_performance_server_timing.html, test_promise_and_timeout_ordering.html:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158897&repo=autoland&lineNumber=2341 - test_WebKitCSSMatrix.html, test_keySync.html:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158875&repo=autoland&lineNumber=6756 - test_bug1436040.html: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158862&repo=autoland&lineNumber=5707
- test_image_crossorigin_data_url.html, test_scroll_space_no_range_overflow_scroll.html, test_readonly.html, test_bug1307853.html:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158960&repo=autoland&lineNumber=6132 - test_animations_variable_changes.html: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=271158952&repo=autoland&lineNumber=2033
Here is the push where you can see the most of them: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=cdfbf9c0a9f1caad870e9fb59f9c1b76a7d0cb41&failure_classification_id=2&selectedJob=271158952
Comment 26•5 years ago
|
||
While we are in soft freeze cycle, can we please hold off the patch re-land after that? Thanks.
Comment 27•5 years ago
|
||
Comment 28•5 years ago
|
||
While we are in soft freeze cycle, can we please hold off the patch re-land after that? Thanks.
Sorry we missed this comment before relanding. Hopefully it isn't a problem to keep this, but if it is we can back out again.
Comment 29•5 years ago
|
||
(In reply to James Graham [:jgraham] from comment #28)
While we are in soft freeze cycle, can we please hold off the patch re-land after that? Thanks.
Sorry we missed this comment before relanding. Hopefully it isn't a problem to keep this, but if it is we can back out again.
No worries. I was thinking about not to (potentially) impact RelEng work for channel merge, especially it got backed out twice that showed some concerns. If try results look good, it's not a problem to keep this. Hope this time we get everything right.
Comment 30•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Updated•5 years ago
|
Description
•