New issue
Advanced search Search tips

Issue 824780 link

Starred by 1 user
Status: WontFix
Owner: ----
Closed: Mar 2018
Cc:
rtoy@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

start operation should have same signature in both AudioScheduledSourceNode and AudioBufferSourceNode

Project Member Reported by terry@google.com, Mar 22 2018 
UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.162 Safari/537.36

Steps to reproduce the problem:
The WebIDL file for

AudioScheduledSourceNode should have a start operation that defined in AudioBufferSourceNode:

    [RaisesException] void start(optional double when = 0, optional double grainOffset, optional double grainDuration);

then the start operation in AudioBufferSourceNode can be removed.

Mozilla has done just that see https://developer.mozilla.org/en-US/docs/Web/API/AudioScheduledSourceNode/start

then AudioBufferSourceNode doesn't need an explicit start operation.

What is the expected behavior?

What went wrong?
WebIDLs should be changed.

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 65.0.3325.162  Channel: n/a
OS Version: Cinnamon
Flash Version:

Comment 1 by terry@google.com, Mar 22 2018 

Should have mentioned that AudioScheduledSourceNode original operation can be removed:

    [RaisesException] void start(optional double when);

Comment 2 by susan.boorgula@chromium.org, Mar 23 2018

Labels: Needs-Triage-M65

Comment 3 by susan.boorgula@chromium.org, Mar 26 2018

Cc: rtoy@chromium.org
Components: Blink>WebAudio
Labels: Triaged-ET TE-NeedsTriageHelp
The issue seems to be out of TE-scope as it is related to AudioScheduledSourceNode and AudioBufferSourceNode.
Hence, adding label TE-NeedsTriageHelp and CC'ing rtoy@ for further help in triaging the issue.

Thanks...!!

Comment 4 by rtoy@chromium.org, Mar 26 2018

Status: WontFix (was: Unconfirmed)
No, the WebAudio spec is pretty clear on this.  Both AudioScheduledSource and AudioBufferSource need to have a start() method.  If it needs to be changed, then it needs to be changed in the spec.

Closing; WAI.
►

Sign in to add a comment