Title: [122253] trunk/Source/WebCore
Revision
122253
Author
[email protected]
Date
2012-07-10 13:21:02 -0700 (Tue, 10 Jul 2012)

Log Message

AudioFIFO not correctly wrapping the write index on push
https://bugs.webkit.org/show_bug.cgi?id=90901

Patch by Raymond Toy <[email protected]> on 2012-07-10
Reviewed by Chris Rogers.

No new tests

* platform/audio/AudioFIFO.cpp:
(WebCore::AudioFIFO::push): Wrap the write index, not read index!

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (122252 => 122253)


--- trunk/Source/WebCore/ChangeLog	2012-07-10 20:14:36 UTC (rev 122252)
+++ trunk/Source/WebCore/ChangeLog	2012-07-10 20:21:02 UTC (rev 122253)
@@ -1,3 +1,15 @@
+2012-07-10  Raymond Toy  <[email protected]>
+
+        AudioFIFO not correctly wrapping the write index on push
+        https://bugs.webkit.org/show_bug.cgi?id=90901
+
+        Reviewed by Chris Rogers.
+
+        No new tests
+
+        * platform/audio/AudioFIFO.cpp:
+        (WebCore::AudioFIFO::push): Wrap the write index, not read index!
+
 2012-07-10  Dana Jansens  <[email protected]>
 
         [chromium] Avoid allocating render pass textures that have no content

Modified: trunk/Source/WebCore/platform/audio/AudioFIFO.cpp (122252 => 122253)


--- trunk/Source/WebCore/platform/audio/AudioFIFO.cpp	2012-07-10 20:14:36 UTC (rev 122252)
+++ trunk/Source/WebCore/platform/audio/AudioFIFO.cpp	2012-07-10 20:21:02 UTC (rev 122253)
@@ -91,7 +91,7 @@
     size_t sourceLength = sourceBus->length();
     size_t part1Length;
     size_t part2Length;
-    findWrapLengths(m_readIndex, sourceLength, part1Length, part2Length);
+    findWrapLengths(m_writeIndex, sourceLength, part1Length, part2Length);
 
     size_t numberOfChannels = m_fifoAudioBus.numberOfChannels();
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to