Closed
Bug 1083657
Opened 10 years ago
Closed 9 years ago
Intermittent browser_projecteditor_delete_file.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/projecteditor/editors.js:248 - TypeError: this.editor is null
Categories
(DevTools :: General, defect)
Tracking
(firefox34 unaffected, firefox35 unaffected, firefox36 fixed, firefox-esr31 unaffected)
RESOLVED
FIXED
Firefox 36
Tracking | Status | |
---|---|---|
firefox34 | --- | unaffected |
firefox35 | --- | unaffected |
firefox36 | --- | fixed |
firefox-esr31 | --- | unaffected |
People
(Reporter: cbook, Assigned: bgrins)
References
()
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
1.55 KB,
patch
|
paul
:
review+
|
Details | Diff | Splinter Review |
Rev4 MacOSX Snow Leopard 10.6 fx-team opt test mochitest-devtools-chrome https://treeherder.mozilla.org/ui/logviewer.html#?job_id=912931&repo=fx-team 16:14:24 INFO - 7825 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/projecteditor/test/browser_projecteditor_delete_file.js | A promise chain failed to handle a rejection: - at resource://gre/modules/commonjs/toolkit/loader.js -> resource:///modules/devtools/projecteditor/editors.js:248 - TypeError: this.editor is null
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Paul, any chance you could get someone to look at this?
Flags: needinfo?(paul)
Comment 16•9 years ago
|
||
Brian, any idea what's going on?
Flags: needinfo?(paul) → needinfo?(bgrinstead)
Assignee | ||
Comment 17•9 years ago
|
||
This should fix it - we had to do a similar thing in the promise resolution in load(). Try push: https://tbpl.mozilla.org/?tree=Try&rev=3a3ba5f62f3c
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Flags: needinfo?(bgrinstead)
Attachment #8527867 -
Flags: review?(paul)
Updated•9 years ago
|
Attachment #8527867 -
Flags: review?(paul) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 18•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/677e3ed1d7cc
Keywords: checkin-needed
Reporter | ||
Comment 19•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/677e3ed1d7cc
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37
Reporter | ||
Updated•9 years ago
|
Target Milestone: Firefox 37 → Firefox 36
Updated•9 years ago
|
status-firefox34:
--- → unaffected
status-firefox35:
--- → unaffected
status-firefox36:
--- → fixed
status-firefox-esr31:
--- → unaffected
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•