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

Categories

(DevTools :: Inspector, defect, P1)

defect

Tracking

(firefox47 fixed)

RESOLVED FIXED
Firefox 47
Tracking Status
firefox47 --- fixed

People

(Reporter: pbro, Assigned: pbro)

Details

Attachments

(1 file)

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.
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+
https://hg.mozilla.org/mozilla-central/rev/76c51477853f
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 47
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.