Closed Bug 1191911 Opened 4 years ago Closed 4 years ago

1,700 instances of "Should've notified blocked=true for a fully finished stream: 'aStream->mNotifiedBlocked'" emitted from dom/media/MediaStreamGraph.cpp during linux64 debug testing

Categories

(Core :: Audio/Video: MediaStreamGraph, defect)

defect
Not set

Tracking

()

RESOLVED INVALID

People

(Reporter: erahm, Unassigned)

References

(Blocks 1 open bug)

Details

This regression occurred somewhere in between 2015/08/04 and 2015/08/06.

> 1703 [NNNNN] WARNING: Should've notified blocked=true for a fully finished stream: 'aStream->mNotifiedBlocked', file dom/media/MediaStreamGraph.cpp, line 354

This warning [1], introduced in bug 1163958 shows up in the following test suites:

> mozilla-central_ubuntu64_vm-debug_test-mochitest-2-bm68-tests1-linux64-build17.txt:854
> mozilla-central_ubuntu64_vm-debug_test-mochitest-e10s-2-bm53-tests1-linux64-build24.txt:834
> mozilla-central_ubuntu64_vm-debug_test-crashtest-bm115-tests1-linux64-build4.txt:11
> mozilla-central_ubuntu64_vm-debug_test-mochitest-browser-chrome-1-bm67-tests1-linux64-build12.txt:4

It shows up in 104 tests. A few of the most prevalent:

> 1044 - dom/media/webaudio/test/test_bug972678.html
> 42 - dom/media/webaudio/test/test_mediaDecoding.html
> 40 - dom/media/webaudio/test/test_stereoPannerNode.html
> 34 - dom/media/webaudio/test/test_periodicWaveBandLimiting.html
> 22 - dom/media/webaudio/test/test_pannerNode_equalPower.html
> 20 - dom/media/webaudio/test/test_channelMergerNodeWithVolume.html
> 20 - dom/media/webaudio/test/test_channelMergerNode.html
> 18 - dom/media/tests/mochitest/test_getUserMedia_gumWithinGum.html
> 12 - dom/media/webaudio/test/test_nodeToParamConnection.html
> 8 - dom/media/webaudio/test/test_waveShaperPassThrough.html

[1] https://hg.mozilla.org/mozilla-central/annotate/b12a261ee32e/dom/media/MediaStreamGraph.cpp#l354
This is the #2 most verbose warning during testing.
Component: Audio/Video → Audio/Video: MSG/cubeb/GMP
This log message has been removed (along with the function)
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.