Closed Bug 1251877 Opened 4 years ago Closed 4 years ago
Enable all devtools/client/inspector/markup/test/browser
_markup _events _jquery* tests with e10s
58 bytes, text/x-review-board-request
All devtools/client/inspector/markup/test/browser_markup_events_jquery* test are disabled with e10s with the following skip-if and comment: skip-if = e10s # Bug 1040751 - CodeMirror editor.destroy() isn't e10s compatible Looks like Bug 1040751 has been closed with WFM, with the reason that editor.destroy seems to work fine with e10s. So I'm assuming these skip-if lines can now be removed safely.
Review commit: https://reviewboard.mozilla.org/r/37031/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/37031/
Attachment #8724453 - Flags: review?(mratcliffe)
Looks like tests pass with e10s on both linux and windows! So that was an easy fix.
Comment on attachment 8724453 [details] MozReview Request: Bug 1251877 - Enable browser_markup_event* tests with e10s; r=miker https://reviewboard.mozilla.org/r/37031/#review33651
Attachment #8724453 - Flags: review?(mratcliffe) → review+
You need to log in before you can comment on or make changes to this bug.