Created attachment 8640905 [details] testcase 1. Load the testcase 2. Open about:memory in a new tab 3. Click "Measure" Assertion failure: NS_IsMainThread(), at AudioNodeEngine.h:348
Karl, that's from you recent cleanup there, right ?
(In reply to Paul Adenot (:padenot) from comment #2) > Karl, that's from you recent cleanup there, right ? Yes, thanks. The assertion was added in https://hg.mozilla.org/mozilla-central/rev/c7ddd289198c#l3.32 Thanks Jesse, for the stacks.
Assignee: nobody → karlt
Created attachment 8641442 [details] [diff] [review] avoid main thread assertion accessing mNode in SizeOfIncludingThis() because the main thread is blocked during the call. These nodes are not owned by the engine, and should really be counted on the main thread, but this is the quick solution without redesigning memory measurement.
Attachment #8641442 - Flags: review?(padenot)
Attachment #8641442 - Flags: review?(padenot) → review+
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox42: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.