1202683 - test_eme_canvas_blocked.html, test_eme_non_mse_fails.html, test_eme_request_notifications.html, test_eme_persistent_sessions.html, test_eme_requestKeySystemAccess.html, test_empty_resource.html causes leaks on Windows debug with runByDir enabled.

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P2)

Description

[Kaustabh Datta Choudhury]

One or more of these tests might be causing leaks in for Windows debug with runByDir enabled. Here is a push to try : https://treeherder.mozilla.org/#/jobs?repo=try&revision=e9475532f009&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-classifiedState=unclassified&filter-searchStr=m%282%29
 
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 3 CondVar (120 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 CrashReporterParent (152 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GMPContentParent (792 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 7 GMPCrashCallback (336 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GMPParent (968 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GMPProcessParent (520 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GMPStorageParent (104 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GMPVideoDecoderParent (152 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GeckoChildProcessHost (480 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GeckoMediaPluginService (96 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 GeckoMediaPluginServiceParent (328 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 6 Mutex (192 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 PCrashReporterParent (48 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 PGMPContentParent (680 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 PGMPParent (672 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 PGMPStorageParent (48 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 PGMPVideoDecoderParent (48 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 ReentrantMonitor (40 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 2 RefCountedMonitor (160 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 4 RefCountedTask (64 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 SharedMemory (32 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 2 StoreRef (32 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 5 WeakReference<MessageListener> (80 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 2 ipc::MessageChannel (768 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 2 nsLocalFile (240 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 27 nsStringBuffer (216 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 28 nsTArray_base (224 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 nsThread (248 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 1 nsTimerImpl (144 bytes)
03:17:04     INFO -  TEST-INFO | leakcheck | default process: leaked 4 nsWeakReference (128 bytes)
03:17:04  WARNING -  TEST-UNEXPECTED-FAIL | leakcheck | default process: 8112 bytes leaked (CondVar, CrashReporterParent, GMPContentParent, GMPCrashCallback, GMPParent, ...)
03:17:04     INFO -  runtests.py | Running tests: end.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

as a note, these tests are disabled:
https://dxr.mozilla.org/mozilla-central/search?q=path%3Adom+1202683&redirect=false&case=true&limit=51&offset=0

:cpearce, is there anything you need from Kaustabh or myself other than the information provided in this bug?

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

Joel: does it look like the leaks are fixed in this Try push?
https://treeherder.mozilla.org/#/jobs?repo=try&revision=060fdf5901bc

If not, how do I repro them on try?

The crash in the "Windows 7 debug" box is already fixed on inbound.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

:cpearce, yes these tests look green and do not show the errors we saw 6 weeks ago.  I am not sure if you have specific patches to help with it, but lets turn them on!

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

Test re-enabled:
https://hg.mozilla.org/integration/mozilla-inbound/rev/48da6c2f495a
(Incorrectly tagged as bug 1209385)