New issue
Advanced search Search tips

Issue 921354 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

AudioWorkletProcessor, process() return value Interpretation is different from specification.

Reported by gait...@gmail.com, Jan 12

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3668.0 Safari/537.36

Steps to reproduce the problem:
Test the EXAMPLE codes on specification.

for example 'BitCrusher' EXAMPLE 16,
there is no process() return value and says

   // No need to return a value; this node’s lifetime is dependent only on its
   // input connections.

but it will not work without 'return true;'

What is the expected behavior?
Nodes that transform their inputs, and are active only while connected inputs and/or script references exist. Such nodes SHOULD return false from process() which allows the presence or absence of connected inputs to determine whether active processing occurs.

What went wrong?
However, in the implementation, even if the input is connected, returning false stops the node.

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 73.0.3668.0  Channel: canary
OS Version: 10.0
Flash Version:
 
Labels: Needs-Triage-M73

Comment 2 by swarnasree.mukkala@chromium.org, Jan 18 (4 days ago)

Cc: swarnasree.mukkala@chromium.org
Labels: Triaged-ET Needs-Feedback
Thanks for filing the issue...

@reporter: Could you please provide a sample test file with the above test-case so that it would be really helpful for triaging the issue.

Comment 3 by hongchan@chromium.org, Jan 18 (4 days ago)

Labels: -Needs-Feedback
Status: Assigned (was: Unconfirmed)
The spec has been changed slightly after we shipped the feature. I will update the code as soon as the AudioWG finishes up the rest of change on Audio Worklet.

Currently I am waiting for:
https://github.com/WebAudio/web-audio-api/issues/1780
https://github.com/WebAudio/web-audio-api/issues/1515

Sign in to add a comment