Skip to content

Commit 69725fe

Browse files
committed
Bug 1872519 use AnalyzeReverseStream() instead of ProcessReverseStream() r=padenot
to clarify that the output from ProcessReverseStream() was unnused. Depends on D198234 Differential Revision: https://phabricator.services.mozilla.com/D198235
1 parent 7d61870 commit 69725fe

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

dom/media/webrtc/MediaEngineWebRTCAudio.cpp

+3-9
Original file line numberDiff line numberDiff line change
@@ -758,15 +758,9 @@ void AudioInputProcessing::ProcessOutputData(MediaTrackGraph* aGraph,
758758
interleavedFarend, framesPerPacketFarend, channelCountFarend,
759759
deinterleavedPacketDataChannelPointers.Elements());
760760

761-
// Having the same config for input and output means we potentially save
762-
// some CPU.
763-
StreamConfig inputConfig(aRate, channelCountFarend);
764-
StreamConfig outputConfig = inputConfig;
765-
766-
// Passing the same pointers here saves a copy inside this function.
767-
DebugOnly<int> err = mAudioProcessing->ProcessReverseStream(
768-
deinterleavedPacketDataChannelPointers.Elements(), inputConfig,
769-
outputConfig, deinterleavedPacketDataChannelPointers.Elements());
761+
StreamConfig reverseConfig(aRate, channelCountFarend);
762+
DebugOnly<int> err = mAudioProcessing->AnalyzeReverseStream(
763+
deinterleavedPacketDataChannelPointers.Elements(), reverseConfig);
770764

771765
MOZ_ASSERT(!err, "Could not process the reverse stream.");
772766
}

0 commit comments

Comments
 (0)