1132370 - Wrong State: Expected 'attached', but current state is 'detached'

Status: RESOLVED FIXED

(DevTools :: Performance Tools (Profiler/Timeline), defect)

Description

[Jordan Santell [:jsantell] [@jsantell]]

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

Comment 1

[Jordan Santell [:jsantell] [@jsantell]]

A few tests have this on gum, and I've seen it once manually working with it.

Comment 2

[Jordan Santell [:jsantell] [@jsantell]]

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'

Comment 3

[Jordan Santell [:jsantell] [@jsantell]]

This is popping up everywhere in tests.

Comment 4

[Victor Porof [:vporof][:vp]]

Right now, this is one of the most prevalent fails on try. Need to fix this!

Comment 5

[Victor Porof [:vporof][:vp]]

s/try/gum/

Comment 6

[Shu-yu Guo [:shu]]

Any progress on this?

Comment 7

[Victor Porof [:vporof][:vp]]

Working on it.

Comment 8

[Victor Porof [:vporof][:vp]]

Attachment: v1

Comment 9

[Jordan Santell [:jsantell] [@jsantell]]

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.

Comment 10

[Victor Porof [:vporof][:vp]]

(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!

Comment 11

[Victor Porof [:vporof][:vp]]

https://hg.mozilla.org/integration/fx-team/rev/9da0a1f0e481

Comment 12

[Victor Porof [:vporof][:vp]]

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.

Comment 13

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/9da0a1f0e481

Comment 14

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8573317 [details] [diff] [review]
v1

Approving for uplift to aurora. Hooray for fixing the tests!

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/38f738759ece