Closed Bug 1055333 Opened 10 years ago Closed 8 years ago

[e10s] Enable devtools/styleeditor tests


(DevTools :: Style Editor, defect)

Not set


(e10s+, firefox46 fixed)

Firefox 46
Tracking Status
e10s + ---
firefox46 --- fixed


(Reporter: bgrins, Assigned: bgrins)



(Whiteboard: [e10s-m7])


(4 files, 1 obsolete file)

      No description provided.
Running tests with mochitest-devtools --e10s browser/devtools/styleeditor/
Part 1 - amazingly only one test seems to be failing when running with e10s, so this moves the skip-if condition to only that test.  Pushed to try:
Attachment #8474880 - Flags: review?(fayearthur)
Comment on attachment 8474880 [details] [diff] [review]

Looks good if try's good.
Attachment #8474880 - Flags: review?(fayearthur) → review+
Keywords: leave-open
Looks like there was an unrelated failure on the try push - here's a new one:
Keywords: checkin-needed
tracking-e10s: --- → +
Blocks: 1058875
I'm seeing quite a few other failures on Holly (, like this:

Though many of them seem to be possibly caused by an error during toolbox destroy that oddly doesn't seem to be an issue with the other test suites):

10:29:35     INFO -  3977 INFO TypeError: this.webconsolePanel is undefined: Toolbox.prototype._saveSplitConsoleHeight@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/framework/toolbox.js:367:1
10:29:35     INFO -  3978 INFO Toolbox.prototype.destroy@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/framework/toolbox.js:1367:5
10:29:35     INFO -  3979 INFO EventEmitter_emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/event-emitter.js:137:11
10:29:35     INFO -  3980 INFO TabTarget.prototype.destroy@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/framework/target.js:447:5
10:29:35     INFO -  3981 INFO TabTarget.prototype.handleEvent@resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/framework/target.js:408:9
10:29:35     INFO -  3982 INFO _beginRemoveTab@chrome://browser/content/tabbrowser.xml:2008:13
10:29:35     INFO -  3983 INFO removeTab@chrome://browser/content/tabbrowser.xml:1894:18
10:29:35     INFO -  3984 INFO removeCurrentTab@chrome://browser/content/tabbrowser.xml:1865:13
10:29:35     INFO -  3985 INFO cleanup@chrome://mochitests/content/browser/browser/devtools/styleeditor/test/head.js:60:5
Looks like the destroy gets called before domReady sets this.webconsolePanel.  Just checks in the destroy function before doing anything to it.  Pushed to try:
Attachment #8479338 - Flags: review?(fayearthur)
Attachment #8479338 - Flags: review?(fayearthur) → review+
Keywords: checkin-needed
Assignee: nobody → bgrinstead
Found a test to skip that I missed the first time around (failures can be seen here:

Pushed to try:
Attachment #8480610 - Flags: review?(fayearthur)
Attachment #8480610 - Flags: review?(fayearthur) → review+
Had an unrelated error in the last try push, here's a new one
Keywords: checkin-needed
Attachment #8474880 - Flags: checkin+
Attachment #8479338 - Flags: checkin+
Attachment #8480610 - Flags: checkin+
Moving DevTools test bugs to e10s milestone M7 (i.e. not blocking e10s merge to Aurora).
Whiteboard: [e10s-m7]
No longer blocks: 1058875
Attached patch styleeditor-e10s-pt4.patch (obsolete) — Splinter Review
Skip remaining tests -- this won't work until Bug 1128988 is fixed (since the cmd_edit test is using a gcli command)
Depends on: 1128988
Summary: Enable devtools/styleeditor tests with e10s → [e10s] Enable devtools/styleeditor tests
Comment on attachment 8601037 [details] [diff] [review]

The e10s dt failure here looks unrelated:  Retriggered a few more just to be sure, but looking like we should be able to close this one out.
Attachment #8601037 - Flags: review?(ejpbruel)
Attachment #8601037 - Flags: review?(ejpbruel) → review+
ni? bgrins because I'm wondering if this isn't easy to land?
Flags: needinfo?(bgrinstead)
Test failures on browser_styleeditor_sourcemap_watching.js in e10s on that push.  Here's a new one that might fix it:
Try looks good.. Checking into fx-team
Flags: needinfo?(bgrinstead)
Keywords: leave-open
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46


Name 			Firefox
Version 		46.0b4
Build ID 		20160322075646
Update Channel 	        beta
User Agent 		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS                      Windows 7 SP1 x86_64

Actual Results: 

Expected Results: 
Unable to perform steps due to STR are not mentioned clearly.
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.