1853630 - use last value of previous event for start of setTargetAtTime() curves

Status: RESOLVED FIXED

(Core :: Web Audio, defect)

Description

[Karl Tomlinson (:karlt)]

From bug 1213313 comment 16:

The start of the curve should actually be obtained from the zeroth tick in the setTarget curve, but the patch here starts the curve one tick prior, as was done before [...]

Comment 1

[Karl Tomlinson (:karlt)]

Attachment: Bug 1853630 use last value of previous event for start of SetTargetAtTime() curves r?padenot

Previously this was happening only after CleanupEventsOlderThan() had been
called between blocks.

t == t0 means the exponential is 1 and the curve uses the value from the curve
of the previous event.
When the time constant is zero and t == t0, the exponential argument is
undefined, but v0 at t0 is the limit as the time constant approaches zero.

Depends on D188436

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/054d3b1a7f96
use last value of previous event for start of SetTargetAtTime() curves r=padenot

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/42058 for changes under testing/web-platform/tests

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/054d3b1a7f96

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot