Bug 1299139 - Don't export symbols from soundtouch on Windows; r=padenot Differential Revision: https://phabricator.services.mozilla.com/D33093

Bug 1444508 remove obsolete expected failures r=padenot These have been lingering since https://hg.mozilla.org/mozilla-central/rev/3ac736416349 due to bug 1474463. Differential Revision: https://phabricator.services.mozilla.com/D34299

Bug 1306999 - Reset mSrcStreamPlaybackEnded when unsetting srcObject. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D33296

Bug 1306999 - Use ProducerIDs in VideoOutput. r=padenot With the extra first-frame-logic in HTMLMediaElement there are two VideoOutput instances feeding frames to the same ImageContainer. To maintain FrameID invariants these need to have unique ProducerIDs. Differential Revision: https://phabricator.services.mozilla.com/D33294

Bug 1542674 - Make Media debug info machine parsable r=padenot,smaug,jya This patch structurizes the media debug information via webidl dictionaries that are returned by HTMLMediaElement::GetMozRequestDebugInfo() and MediaSource::GetMozDebugReaderData(). Differential Revision: https://phabricator.services.mozilla.com/D27893

Bug 1121623 part 4. Remove AudioScheduledSourceNode::WrapObject. r=padenot It's dead code, because AudioScheduledSourceNode is an abstract class and all subclasses override WrapObject. Differential Revision: https://phabricator.services.mozilla.com/D32202

Bug 1552194 - Consider ftyp 3gp4 as being mp4. r=padenot 3gp4 is based on mp4. Should the codecs not be supported we will simply error later. Differential Revision: https://phabricator.services.mozilla.com/D31919

bug 1548816 crashtest r=padenot Differential Revision: https://phabricator.services.mozilla.com/D30160

Bug 1552652 - Unbreak building tests on Tier3 after bug 1545079. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D31756

Bug 1545079 - Create a gtest that exercises revive method of AudioCallbackDriver. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D30890

Bug 1545079 - Execute revive logic in operation thread. r=padenot `IsStarted` may not be updated at the time `Revive()` method is executed since the update of the flag happens on the async operation. This will be the case if `Revive` is executed right after `Start`. When that happens the revive method asserts that the stream is started. Differential Revision: https://phabricator.services.mozilla.com/D30889

Bug 1545079 - Enhance MockCubeb class to simulate a cubeb stream. r=padenot On MochCubeb add a fake audio thread and the corresponding methods for stream_{init,start,stop,destroy}. Differential Revision: https://phabricator.services.mozilla.com/D30888

Bug 1545079 - Move MockCubeb on a seperate reusable header. r=padenot Extract the existing MockCubeb logic from TestAudioDeviceEnumerator to a separate reusable header. Differential Revision: https://phabricator.services.mozilla.com/D30887

Bug 1552342 - Update libcubeb to b9e2c50. r=padenot

Bug 1546865 - Retry gUM on mic process limit collision after 200 ms. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D30290

Bug 1548816 be consistent about number of frames copied r=padenot Differential Revision: https://phabricator.services.mozilla.com/D30159

Bug 1542674 - Make Media debug info machine parsable r=padenot,smaug This patch structurizes the media debug information via webidl dictionaries that are returned by HTMLMediaElement::GetMozRequestDebugInfo() and MediaSource::GetMozDebugReaderData(). Differential Revision: https://phabricator.services.mozilla.com/D27893

Bug 1547381 - Call StopTrack() in the right order. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D29924

Bug 1546655 - Skip 'flac-s24.flac' and 'opus-mapping2.webm' in test_streams_element_capture.html r=padenot Differential Revision: https://phabricator.services.mozilla.com/D29594

Bug 1530178 - Update WPT expectations. r=padenot https://github.com/WebAudio/web-audio-api/issues/1004 Differential Revision: https://phabricator.services.mozilla.com/D26474

Bug 1530178 - copyFromChannel/copyToChannel until end of buffer when destination/source is longer r=padenot,karlt than available space, as clarified in https://github.com/WebAudio/web-audio-api/issues/1004 Differential Revision: https://phabricator.services.mozilla.com/D25791

Bug 1546872 - Update cubeb from upstream to 64aa80f. r=padenot Pick commit: 64aa80f - wasapi: notify when a device is disabled. BMO 1546872

Bug 1545133 - Avoid trying to double-suspend a MediaStream if external code makes the suspendCount negative. r=padenot We don't currently know any ways in which external code can make the suspendCount negative, but this will keep us safe should a future bug enable it. Depends on D28805 Differential Revision: https://phabricator.services.mozilla.com/D28806

Bug 1545133 - Add crashtest. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D28805

Bug 1545279 - Update cubeb from upstream to 3570749. r=padenot Pick commits: 3570749 - wasapi: tie monitor lifetime with notification client lifetime. BMO 1545279 (#505) 162625a - test: add option to get the posotion of a stream (#504) Differential Revision: https://phabricator.services.mozilla.com/D28796

Bug 1546655 - Fix test filtering in test_streams_element_capture.html. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D28666

Bug 1546655 - Skip ambisonics.mp4 in test_streams_element_capture.html. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D28651

Bug 1544650 - Always pre-create MediaStreamTracks for DecodedStream in MediaDecoder. r=padenot This moves the responsibility for creating MediaStreamTracks from DecodedStream::Start to MediaDecoder. This let's MediaDecoder create them as soon as metadata is known. This gives the application guarantees on when tracks can be expected to exist, and aligns with the spec that says they should be created when metadata is known. Differential Revision: https://phabricator.services.mozilla.com/D28473

Bug 1544650 - Add mochitest. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D28472

Bug 1536766 - Update WPT expectations. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D28604

Bug 1536766 - End a track only after the graph has reported reaching its end time in DecodedStream. r=jya,padenot This gives us a guarantee that the first frame of a media file can be rendered with a second media element and mozCaptureStream(), even if the file is very very short. With longer video-only files there were also cases where nothing ended up being rendered. Probably because the MediaStreamGraph ended up switching from an AudioCallbackDriver to a SystemClockDriver and this took enough time to put the SourceMediaStream::EndTrack and the SourceMediaStream::AddTrackListener calls for this video track to be processed in the same iteration. The listener would then always lose to the ending track and update main thread state too late, leading to its media element not rendering any frames and nasty intermittent failures. Differential Revision: https://phabricator.services.mozilla.com/D27270

Bug 1536766 - Drop frames more accurately when ending and interrupting video track playback. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D27269

Bug 1536766 - Make NotifyOutput from SourceMediaStream tracks reflect amount of data in the track. r=padenot `streamCurrentTime` is relative to the start of the stream, as is the track's segment. We need to account for the track's start time in the stream when notifying of output. Differential Revision: https://phabricator.services.mozilla.com/D27268

Bug 1536766 - Remove unused MediaStream::HasCurrentData. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D27266

Bug 1536766 - Handle DecodedStream's resets when calculating VideoChunk durations in the MSG. r=padenot This is necessary to get an accurate total track duration in case DecodedStream does a lot of seeking and pausing. This also removes previous code for avoiding blocking a video track, as this new code handles that case too. Differential Revision: https://phabricator.services.mozilla.com/D27264

Bug 1536766 - Don't ignore null frames in MediaStreamGraph. r=padenot Null frames could still be valid. It's how DecodedStream signals a pause. They could also be valid and have an intrinsic size and the force-black flag set. Differential Revision: https://phabricator.services.mozilla.com/D27263

Bug 1544023 - let AudioNode keep the reference of all AudioParams which belong to itself. r=padenot When we suspend or resume the `AudioContext`, it should affect ALL media streams which belong to or are related to the `AudioNode` that are created by this `AudioContext`. As `AudioNode::OutputParams()` can only return the connected AudioParams, it doesn't return the AudioParams which are belong to itself. That means we would miss to apply the suspend/resume operation for those streams, and it would cause imbalancing suspended count. Therefore, we let `AudioNode` to keep the reference of all its AudioParam, and return them to `AudioContext` in order to do the operation for all streams. Differential Revision: https://phabricator.services.mozilla.com/D28008

Bug 1546012 - Avoid slow sleeps on Android in TestSPSCQueue; r=padenot Reducing the sleep to 0 on Android results in a dramatic reduction in run time, and the test passes consistently. Differential Revision: https://phabricator.services.mozilla.com/D28296

Bug 1541290 - Restart driver properly on revive method. r=padenot Revive method of AudioCallbackDriver was wrong because it was re-initializing an already initialized driver. That was hitting an assert. Instead of that it should stop the drained stream and start it again. Also, mStarted member should be reset properly on the stop method. Differential Revision: https://phabricator.services.mozilla.com/D26678

Bug 1536766 - End a track only after the graph has reported reaching its end time in DecodedStream. r=jya,padenot This gives us a guarantee that the first frame of a media file can be rendered with a second media element and mozCaptureStream(), even if the file is very very short. With longer video-only files there were also cases where nothing ended up being rendered. Probably because the MediaStreamGraph ended up switching from an AudioCallbackDriver to a SystemClockDriver and this took enough time to put the SourceMediaStream::EndTrack and the SourceMediaStream::AddTrackListener calls for this video track to be processed in the same iteration. The listener would then always lose to the ending track and update main thread state too late, leading to its media element not rendering any frames and nasty intermittent failures. Differential Revision: https://phabricator.services.mozilla.com/D27270

Bug 1536766 - Drop frames more accurately when ending and interrupting video track playback. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D27269

Bug 1536766 - Make NotifyOutput from SourceMediaStream tracks reflect amount of data in the track. r=padenot `streamCurrentTime` is relative to the start of the stream, as is the track's segment. We need to account for the track's start time in the stream when notifying of output. Differential Revision: https://phabricator.services.mozilla.com/D27268

Bug 1536766 - Remove unused MediaStream::HasCurrentData. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D27266

Bug 1536766 - Handle DecodedStream's resets when calculating VideoChunk durations in the MSG. r=padenot This is necessary to get an accurate total track duration in case DecodedStream does a lot of seeking and pausing. This also removes previous code for avoiding blocking a video track, as this new code handles that case too. Differential Revision: https://phabricator.services.mozilla.com/D27264

Bug 1536766 - Don't ignore null frames in MediaStreamGraph. r=padenot Null frames could still be valid. It's how DecodedStream signals a pause. They could also be valid and have an intrinsic size and the force-black flag set. Differential Revision: https://phabricator.services.mozilla.com/D27263

Bug 1543032 - Extend metrics for dropped frames - r=padenot This patch adds the number of dropped frames for each step of the process (read/sink/compositor) and gives us more insight about where frames are dropped, as opposed to the getVideoPlaybackQuality() API which gives the grand total. Differential Revision: https://phabricator.services.mozilla.com/D27488

Bug 1542685 - Avoid integer overflows by multiplying doubles. r=padenot SaferMultDiv(time, audioScale, videoScale) could easily result in overflow because all three args are roughly equal, and SaferMultDiv would always do the multiplication first. The worst-case is then multiplying an int64_t to another int64_t that have very similar values. Since we represent time here in microseconds, this would overflow after only 50 minutes. Differential Revision: https://phabricator.services.mozilla.com/D26494

Bug 1541311 add support for AudioWorkletNode.numberOfInputs/Outputs r=padenot There is no specified limit on the number of inputs or outputs, except that the webidl parameter is unsigned long, but Gecko has an implementation-defined limit. Differential Revision: https://phabricator.services.mozilla.com/D25898

Bug 1539039 shutdown worklet from AudioContext::Shutdown() r=padenot On process shutdown, the MSG is shut down (and so are other threads) before the final main thread cycle collection, and so we can't depend on unlink to notify the worklet before MSG shutdown. Depends on D25351 Differential Revision: https://phabricator.services.mozilla.com/D25352

Bug 1539039 finish worklet before destination stream is destroyed r=padenot AudioDestinationNode, AudioContext, and Worklet are part of the same cycle. Any of these may be unlinked before another. The unlink process for this cycle is the notification that the worklet will no longer be used, but the worklet uses the destination stream to send a message to release graph thread objects and so must do this before the stream is destroyed. Differential Revision: https://phabricator.services.mozilla.com/D25351