[Webkit-unassigned] [Bug 261066] New: Handle NaN in the Audio delay curves

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=261066

            Bug ID: 261066
           Summary: Handle NaN in the Audio delay curves
           Product: WebKit
           Version: Safari 16
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Web Audio
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ahmad.saleem792 at gmail.com
                CC: cdumez at apple.com

Hi Team,

Based on Slack's discussion with Chris 1:1, it is potential merge from Blink:

Blink Commit: https://chromium.googlesource.com/chromium/src.git/+/f6e3d4665d9261e4fef2b5931e4c75ecb5e032bf

WebKit Source: https://searchfox.org/wubkat/source/Source/WebCore/Modules/webaudio/DelayDSPKernel.cpp#124

______

It is fixed by following:

        double delayTime = m_delayTimes[i];
        if (std::isnan(delayTime))
            delayTime = maxDelayTime();
        else
            delayTime = std::clamp<double>(m_delayTimes[i], 0.0, maxDelayTime());

____

Raising this bug so we can fix it in WebKit as well.

Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230902/f5cebfcc/attachment.htm>