832121 - browser_profiler_controller.js appears to never stop the profiler

Status: RESOLVED DUPLICATE of bug 830990

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

Description

[Marco Bonardo [:mak]]

this is what I see in a log

TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | isActive didn't return any errors
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | Profiler is not active
MOZ_EVENT_TRACE start 1358455604669
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | Profiler started without errors
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | isActive didn't return any errors
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | Profiler is active
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | Profiler stopped without errors
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | Profiler returned some data
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | isActive didn't return any errors
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | Profiler is not active
INFO TEST-END | chrome://mochitests/content/browser/browser/devtools/profiler/test/browser_profiler_controller.js | finished in 496ms

Then I see a bunch of
MOZ_EVENT_TRACE sample XXXX
repeated along all of the following tests

and after all of the tests run

INFO TEST-START | Shutdown
Browser Chrome Test Summary
	Passed: 34312
	Failed: 12
	Todo: 38

*** End BrowserChrome Test Results ***
MOZ_EVENT_TRACE sample 1358456227746 818
MOZ_EVENT_TRACE stop 1358456227746

I'm not sure if this is expected (a test buffer error?) but looks like if it's really enabled along all the tests it may cause issues

Comment 1

[Anton Kovalyov (:anton)]

Stopping profiler doesn't turn the event tracer off. I filed a bug about it a couple of days ago: bug 830990.