1458045 - Strange error message from exponentialRampToValueAtTime

Status: RESOLVED FIXED

(Core :: Web Audio, defect, P2)

Description

[toy.raymond]

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.117 Safari/537.36

Steps to reproduce:

In console:

c = new AudioContext()
g = new GainNode(c)
g.gain.exponentialRampToValueAtTime(-2, 5)


Actual results:

Error signaled:

SyntaxError: An invalid or illegal string was specified


Expected results:

I was expecting this to either work, or at least a different error message.

It's not illegal (anymore) to give an negative value for exponentialRampToValue().  There are some constraints, however in the spec: 
https://webaudio.github.io/web-audio-api/#dom-audioparam-exponentialramptovalueattime

Comment 1

[Cristian Craciun]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0
Build ID: 20180503100146

I have managed to reproduce this issue on the latest FF release (59) and the latest Nightly build 61.0a1 (2018-05-03). 

The issue occurs regardless of the OS used. 

Based on the description and the tests executed, I will mark this bug as NEW.

Comment 2

[Karl Tomlinson (:karlt)]

Attachment: Bug 1458045 use V0 for exponentialRampToValueAtTime() curve when V0 and V1 have opposite signs or V0 is zero r?padenot

https://github.com/WebAudio/web-audio-api/pull/624

Depends on D189101

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/52f0910d6e37
use V0 for exponentialRampToValueAtTime() curve when V0 and V1 have opposite signs or V0 is zero r=padenot

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/52f0910d6e37

Comment 5

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

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

Comment 6

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

Upstream PR merged by moz-wptsync-bot