test_texttrackcue_chrome.html leaks from mochitest-chrome when running dom/media/test/ as a standalone directory

RESOLVED WORKSFORME

Status

()

RESOLVED WORKSFORME
4 years ago
3 years ago

People

(Reporter: jmaher, Unassigned)

Tracking

Trunk
x86_64
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
when running this directory as a standalone directory on mac osx 10.8 (and only 10.8) we get a leak:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=02eab22cbe92

05:53:14 INFO - |<----------------Class--------------->|<-----Bytes------>|<----------------Objects---------------->|<--------------References-------------->|
05:53:14 INFO - Per-Inst Leaked Total Rem Mean StdDev Total Rem Mean StdDev
05:53:14 INFO - 0 TOTAL 31 2331 847436 45 ( 7160.60 +/- 11229.36) 1534781 5 ( 4490.38 +/- 8468.26)
05:53:14 INFO - 17 AsyncLatencyLogger 72 72 1 1 ( 1.00 +/- 0.00) 7 1 ( 3.46 +/- 1.61)
05:53:14 INFO - 29 AudioNodeEngine 56 56 3 1 ( 1.80 +/- 0.84) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 30 AudioNodeStream 456 456 3 1 ( 1.80 +/- 0.84) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 114 CondVar 40 80 121 2 ( 56.42 +/- 32.68) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 120 ControlMessage 16 16 11 1 ( 4.81 +/- 3.04) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 198 GraphDriver 88 88 1 1 ( 1.00 +/- 0.00) 3 1 ( 1.80 +/- 0.84)
05:53:14 INFO - 277 MediaSegment 24 24 2 1 ( 1.33 +/- 0.58) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 278 MediaStream 352 704 6 2 ( 3.50 +/- 1.58) 10 2 ( 3.94 +/- 1.70)
05:53:14 INFO - 279 MediaStreamGraph 32 32 1 1 ( 1.00 +/- 0.00) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 280 MediaStreamGraphImpl 416 416 1 1 ( 1.00 +/- 0.00) 7 1 ( 1.77 +/- 0.73)
05:53:14 INFO - 290 Mutex 32 128 754 4 ( 196.19 +/- 103.94) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 430 StreamBuffer 40 40 3 1 ( 1.80 +/- 0.84) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 456 TimeVaryingBase 1 3 9 3 ( 5.20 +/- 2.37) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 460 Track 24 24 1 1 ( 1.00 +/- 0.00) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - 1037 nsTArray_base 8 192 176905 24 (25463.18 +/- 10308.67) 0 0 ( 0.00 +/- 0.00)
05:53:14 INFO - nsTraceRefcnt::DumpStatistics: 1159 entries
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 AsyncLatencyLogger (72 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 AudioNodeEngine (56 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 AudioNodeStream (456 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 2 CondVar (80 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 ControlMessage (16 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 GraphDriver (88 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 MediaSegment (24 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 2 MediaStream (704 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 MediaStreamGraph (32 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 MediaStreamGraphImpl (416 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 4 Mutex (128 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 StreamBuffer (40 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 3 TimeVaryingBase (3 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 1 Track (24 bytes)
05:53:14 INFO - TEST-INFO | leakcheck | default process: leaked 24 nsTArray_base (192 bytes)
05:53:14 WARNING - TEST-UNEXPECTED-FAIL | leakcheck | default process: 2331 bytes leaked (AsyncLatencyLogger, AudioNodeEngine, AudioNodeStream, CondVar, ControlMessage, ...)
(Reporter)

Comment 1

4 years ago
Rick, can you take a first look at this and see why this might be leaking on osx 10.8.
Flags: needinfo?(rick.eyre)
Summary: test_texttrackcue_chrome.html leaks from mochitst-chrome when running dom/media/test/ as a standalone directory → test_texttrackcue_chrome.html leaks from mochitest-chrome when running dom/media/test/ as a standalone directory
This might be related to bug 1116202.
(Reporter)

Comment 3

4 years ago
Thanks Andrew- any thing for me to try out?  I am willing to hack this a bit, but would like to get to the bottom of this faster as I was planning on landing my run-by-dir changes for mochitest-chrome on Monday.

*I blame try server for not running 10.8 by default, it always catches me right before or after I land a change :)
I don't know much about that bug, except that it involves MediaStreamGraph shutdown.

Comment 5

4 years ago
Sorry Joel, I don't really have much time to help out these days!
Flags: needinfo?(rick.eyre)

Comment 6

4 years ago
Not too sure who else is working on the Track stuff at the moment either.
Component: Audio/Video → Audio/Video: Playback
We no longer test on 10.8, so I don't know if it's fixed, but the leak is no longer an issue. I've re-enabled the test in bug 1253762.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.