AudioParam.value setter should call setValueAtTime(value, now), as per: - https://www.w3.org/TR/webaudio/#dom-audioparam-value
Created attachment 407498 [details] Patch
Comment on attachment 407498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=407498&action=review > Source/WebCore/Modules/webaudio/AudioParam.cpp:94 > + setValue(value); > + auto result = setValueAtTime(m_value, context().currentTime()); Why call both setValue and setValueAtTime?
Committed r266293: <https://trac.webkit.org/changeset/266293> All reviewed patches have been landed. Closing bug and clearing flags on attachment 407498 [details].
Comment on attachment 407498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=407498&action=review >> Source/WebCore/Modules/webaudio/AudioParam.cpp:94 >> + auto result = setValueAtTime(m_value, context().currentTime()); > > Why call both setValue and setValueAtTime? I guess we are using setValue to clamp the value for us.
<rdar://problem/67961959>
Comment on attachment 407498 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=407498&action=review >>> Source/WebCore/Modules/webaudio/AudioParam.cpp:94 >>> + auto result = setValueAtTime(m_value, context().currentTime()); >> >> Why call both setValue and setValueAtTime? > > I guess we are using setValue to clamp the value for us. I’m really unclear on whether this is intentional and correct or not. And don’t understand the tests well enough to know if it’s tested.