Changeset 94969 in webkit


Ignore:
Timestamp:
Sep 12, 2011 11:55:07 AM (13 years ago)
Author:
crogers@google.com
Message:

DelayNode delay buffer is not correctly wrapping around
​https://bugs.webkit.org/show_bug.cgi?id=67872

Reviewed by Kenneth Russell.

Source/WebCore:

Test: webaudio/delaynode.html

  • webaudio/DelayDSPKernel.cpp:

(WebCore::DelayDSPKernel::process):

LayoutTests:

  • webaudio/delaynode-expected.txt: Added.
  • webaudio/delaynode.html: Added.
Location:
trunk
Files:
2 added
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r94966 r94969  
     12011-09-12  Chris Rogers  <crogers@google.com>
     2
     3        DelayNode delay buffer is not correctly wrapping around
     4        https://bugs.webkit.org/show_bug.cgi?id=67872
     5
     6        Reviewed by Kenneth Russell.
     7
     8        * webaudio/delaynode-expected.txt: Added.
     9        * webaudio/delaynode.html: Added.
     10
    1112011-09-12  Arko Saha  <arko@motorola.com>
    212

  • trunk/Source/WebCore/ChangeLog

    r94966 r94969  
     12011-09-12  Chris Rogers  <crogers@google.com>
     2
     3        DelayNode delay buffer is not correctly wrapping around
     4        https://bugs.webkit.org/show_bug.cgi?id=67872
     5
     6        Reviewed by Kenneth Russell.
     7
     8        Test: webaudio/delaynode.html
     9
     10        * webaudio/DelayDSPKernel.cpp:
     11        (WebCore::DelayDSPKernel::process):
     12
    1132011-09-12  Arko Saha  <arko@motorola.com>
    214

  • trunk/Source/WebCore/webaudio/DelayDSPKernel.cpp

    r92408 r94969  
    109109
    110110        double readPosition = m_writeIndex + bufferLength - desiredDelayFrames;
    111         if (readPosition > bufferLength)
     111        if (readPosition >= bufferLength)
    112112            readPosition -= bufferLength;
    113113
Note: See TracChangeset for help on using the changeset viewer.