[e10s] Enable browser_webconsole_closure_inspection.js in e10s

RESOLVED FIXED in Firefox 48

Status

defect
P1
normal
RESOLVED FIXED
4 years ago
Last year

People

(Reporter: linclark, Assigned: bgrins)

Tracking

(Blocks 1 bug)

Trunk
Firefox 48
Dependency tree / graph

Firefox Tracking Flags

(e10s+, firefox47 affected, firefox48 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

4 years ago
No description provided.
Reporter

Updated

4 years ago
Blocks: 1112599
Blocks: e10s-tests
tracking-e10s: --- → +
Assignee

Updated

3 years ago
Priority: -- → P1
Assignee

Comment 1

3 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9eea38922ec6
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Reporter

Updated

3 years ago
Attachment #8733087 - Flags: review?(lclark) → review+
Reporter

Comment 3

3 years ago
Comment on attachment 8733087 [details]
MozReview Request: Bug 1243982 - Enable browser_webconsole_closure_inspection.js in e10s;r=linclark

https://reviewboard.mozilla.org/r/41543/#review38185

::: devtools/client/webconsole/test/browser_webconsole_closure_inspection.js:32
(Diff revision 1)
>    }
>  
>    loadTab(TEST_URI).then(() => {
>      openConsole().then((hud) => {
>        openDebugger().then(({ toolbox, panelWin }) => {
>          let deferred = promise.defer();

Might want to switch this to Promise (unless deferred is required in this instance). But I also know that we want to enable these tests quickly, so ok to commit as is.

Comment 5

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/00b9f30c26f4
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48

Updated

Last year
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.