Closed
Bug 1132370
Opened 10 years ago
Closed 10 years ago
Wrong State: Expected 'attached', but current state is 'detached'
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect)
Tracking
(firefox38 fixed, firefox39 fixed)
RESOLVED
FIXED
Firefox 39
People
(Reporter: jsantell, Assigned: vporof)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jsantell
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
4757 INFO TEST-UNEXPECTED-FAIL | browser/devtools/performance/test/browser_perf-details-memory-calltree-render.js | A promise chain failed to handle a rejection: - Protocol error (unknownError): Error: Wrong State: Expected 'attached', but current state is 'detached' Only saw this once on gum on linux debug https://treeherder.mozilla.org/#/jobs?repo=gum&revision=c87fcfad32e4
Reporter | ||
Updated•10 years ago
|
Blocks: perf-tool-v2
Reporter | ||
Updated•10 years ago
|
Summary: INFO TEST-UNEXPECTED-FAIL browser/devtools/performance/test/browser_perf-details-memory-calltree-render.js → Wrong State: Expected 'attached', but current state is 'detached'
Reporter | ||
Comment 1•10 years ago
|
||
A few tests have this on gum, and I've seen it once manually working with it.
Reporter | ||
Comment 2•10 years ago
|
||
4814 INFO TEST-UNEXPECTED-FAIL | browser/devtools/performance/test/browser_perf-overview-render-03.js | A promise chain failed to handle a rejection: - Protocol error (unknownError): Error: Wrong State: Expected 'attached', but current state is 'detached' 4873 INFO TEST-UNEXPECTED-FAIL | browser/devtools/performance/test/browser_perf-refresh.js | A promise chain failed to handle a rejection: - Protocol error (unknownError): Error: Wrong State: Expected 'attached', but current state is 'detached'
Reporter | ||
Updated•10 years ago
|
Blocks: enable-perf-tool
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 4•10 years ago
|
||
Right now, this is one of the most prevalent fails on try. Need to fix this!
Assignee | ||
Comment 8•10 years ago
|
||
Reporter | ||
Comment 9•10 years ago
|
||
Comment on attachment 8573317 [details] [diff] [review] v1 Review of attachment 8573317 [details] [diff] [review]: ----------------------------------------------------------------- Awesome. Is this worth an uplift to aurora? There's no b2g version based on fx38, but I think that will be an ESR version.
Attachment #8573317 -
Flags: review?(jsantell) → review+
Assignee | ||
Comment 10•10 years ago
|
||
(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #9) > Comment on attachment 8573317 [details] [diff] [review] > v1 > > Review of attachment 8573317 [details] [diff] [review]: > ----------------------------------------------------------------- > > Awesome. Is this worth an uplift to aurora? There's no b2g version based on > fx38, but I think that will be an ESR version. Yeah, sure!
Assignee | ||
Comment 11•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/9da0a1f0e481
Whiteboard: [fixed-in-fx-team]
Assignee | ||
Comment 12•10 years ago
|
||
Comment on attachment 8573317 [details] [diff] [review] v1 Approval Request Comment [Feature/regressing bug #]: Fixes an intermittent test failure for a (yet to be enabled by default) developer tool. [User impact if declined]: Possible backwards incompatibility between the devtools server and client. [Describe test coverage new/current, TreeHerder]: This fix affects all the tests. [Risks and why]: No risk. [String/UUID change made/needed]: None.
Attachment #8573317 -
Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/9da0a1f0e481
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 39
Comment on attachment 8573317 [details] [diff] [review] v1 Approving for uplift to aurora. Hooray for fixing the tests!
Attachment #8573317 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 15•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/38f738759ece
status-firefox38:
--- → fixed
Keywords: checkin-needed
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•