corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1580659 - part6 : remove input config parameter from 'NotifyStartedPlaying()'. r=Ehsan

Browse Source 
After applying patch4, now we would pull the change from `AudioChannelService` everytime after starting the agent, so we don't need to rely on letting `NotifyStartedPlaying()` to modify the config and use it to update our state.

Differential Revision: https://phabricator.services.mozilla.com/D45753
This commit is contained in:
alwu
2019-09-24 21:18:07 +00:00
parent 2d2696928c
commit 47461f8a6b
6 changed files with 16 additions and 42 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
dom/html/HTMLMediaElement.cpp
View File

@@ -1187,9 +1187,8 @@ class HTMLMediaElement::AudioChannelAgentCallback final
void StartAudioChannelAgent() {
MOZ_ASSERT(mAudioChannelAgent);
MOZ_ASSERT(!mAudioChannelAgent->IsPlayingStarted());
AudioPlaybackConfig config;
if (NS_WARN_IF(NS_FAILED(mAudioChannelAgent->NotifyStartedPlaying(
&config, IsOwnerAudible())))) {
if (NS_WARN_IF(NS_FAILED(
mAudioChannelAgent->NotifyStartedPlaying(IsOwnerAudible())))) {
return;
}