1132206 - CRASH: browser_perf-details-03.js | application crashed

Status: RESOLVED FIXED

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

Description

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

After the test runs, it hangs on wrapping up and beachballs.

35 INFO TEST-PASS | browser/devtools/performance/test/browser_perf-details-03.js | The waterfall view is now selected when toggling off enable-memory when a memory panel is selected. 
36 INFO Destroying the performance tool.
37 INFO Removing tab.
38 INFO Tab removed and finished closing.
###!!! [Parent][OnMaybeDequeueOne] Error: Channel closing: too late to send/recv, messages will be lost
###!!! [Parent][OnMaybeDequeueOne] Error: Channel closing: too late to send/recv, messages will be lost

[HANGS]

Comment 1

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

Crashing on gum too, will see if patch on gum that disables current profiler in bug 1123815  fixes it, if not, will disable.


4717 ERROR TEST-UNEXPECTED-TIMEOUT | browser/devtools/performance/test/browser_perf-details-03.js | application timed out after 330 seconds with no output on browser/devtools/performance/test
TEST-UNEXPECTED-FAIL | ShutdownLeaks | process() called before end of test suite
4718 ERROR TEST-UNEXPECTED-FAIL | browser/devtools/performance/test/browser_perf-details-03.js | application terminated with exit code 5
PROCESS-CRASH | browser/devtools/performance/test/browser_perf-details-03.js | application crashed [@ 0x14e889a2d]

Comment 2

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

Attachment: 1132206-disable.patch

Let's get this in gum/fx-team so we can see if any other failures are occuring.

Comment 3

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

Comment on attachment 8563157 [details] [diff] [review]
1132206-disable.patch

Review of attachment 8563157 [details] [diff] [review]:
-----------------------------------------------------------------

:(

Comment 4

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

Landed on gum: https://hg.mozilla.org/projects/gum/rev/d26b27edc045

Needs to be landed on fx-team too once the tree opens.

Comment 5

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

Landed on fx-team: https://hg.mozilla.org/integration/fx-team/rev/ca851166b6fb
Need to merge.

Comment 6

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

Merged: https://treeherder.mozilla.org/#/jobs?repo=gum&revision=c87fcfad32e4

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ca851166b6fb

Comment 8

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

Very likely fixed by bug 1137503

Comment 9

[Firefox Bug Husbandry Bot]

Removing leave-open keyword from resolved bugs, per :sylvestre.