Title: [216954] trunk/Source/WebCore
- Revision
- 216954
- Author
- eric.carl...@apple.com
- Date
- 2017-05-16 16:10:45 -0700 (Tue, 16 May 2017)
Log Message
[MediaStream] AudioSampleBufferList::zeroABL takes byte count, not sample count
https://bugs.webkit.org/show_bug.cgi?id=172194
<rdar://problem/32233799>
Reviewed by Jer Noble.
* platform/mediastream/mac/AudioTrackPrivateMediaStreamCocoa.cpp:
(WebCore::AudioTrackPrivateMediaStreamCocoa::render): Pass number of bytes to zero,
not number of samples.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (216953 => 216954)
--- trunk/Source/WebCore/ChangeLog 2017-05-16 23:06:53 UTC (rev 216953)
+++ trunk/Source/WebCore/ChangeLog 2017-05-16 23:10:45 UTC (rev 216954)
@@ -1,3 +1,15 @@
+2017-05-16 Eric Carlson <eric.carl...@apple.com>
+
+ [MediaStream] AudioSampleBufferList::zeroABL takes byte count, not sample count
+ https://bugs.webkit.org/show_bug.cgi?id=172194
+ <rdar://problem/32233799>
+
+ Reviewed by Jer Noble.
+
+ * platform/mediastream/mac/AudioTrackPrivateMediaStreamCocoa.cpp:
+ (WebCore::AudioTrackPrivateMediaStreamCocoa::render): Pass number of bytes to zero,
+ not number of samples.
+
2017-05-16 Mark Lam <mark....@apple.com>
WorkerRunLoop::Task::performTask() needs to null check context->script() before use.
Modified: trunk/Source/WebCore/platform/mediastream/mac/AudioTrackPrivateMediaStreamCocoa.cpp (216953 => 216954)
--- trunk/Source/WebCore/platform/mediastream/mac/AudioTrackPrivateMediaStreamCocoa.cpp 2017-05-16 23:06:53 UTC (rev 216953)
+++ trunk/Source/WebCore/platform/mediastream/mac/AudioTrackPrivateMediaStreamCocoa.cpp 2017-05-16 23:10:45 UTC (rev 216954)
@@ -226,7 +226,7 @@
Ref<AudioTrackPrivateMediaStreamCocoa> protectedThis { *this };
if (!m_isPlaying || m_muted || !m_dataSource || streamTrack().muted() || streamTrack().ended() || !streamTrack().enabled()) {
- AudioSampleBufferList::zeroABL(ioData, static_cast<size_t>(sampleCount));
+ AudioSampleBufferList::zeroABL(ioData, static_cast<size_t>(sampleCount * m_outputDescription->bytesPerFrame()));
actionFlags = kAudioUnitRenderAction_OutputIsSilence;
return 0;
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes