Closed
Bug 1007776
Opened 11 years ago
Closed 10 years ago
Add a devtools API for muting the source AudioNodes
Categories
(Core :: Web Audio, defect)
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: jsantell, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file)
13.39 KB,
patch
|
roc
:
review+
smaug
:
review+
|
Details | Diff | Splinter Review |
AudioSourceNode's should be able to be muted.
Assignee | ||
Comment 1•10 years ago
|
||
The underlying code that I'm adding in bug 1007778 may help with muting the source nodes as well...
Assignee: nobody → ehsan
Component: Developer Tools: Web Audio Editor → Web Audio
Product: Firefox → Core
Summary: Muting AudioNode's with web audio editor → Add a devtools API for muting the source AudioNodes
Assignee | ||
Comment 2•10 years ago
|
||
FWIW I'm going to add passThrough attributes for the source nodes as well, but I'll wait until I get r+ on bug 1007778 before I write a patch for this. You can use the same API on your side now, FWIW, if you take my experimental patch and add the necessary *.webidl chunks for AudioBufferSourceNode, MediaElement/StreamAudioSourceNode and OscillatorNode.
Depends on: 1007778
Assignee | ||
Comment 3•10 years ago
|
||
(semantically you can imagine that the source nodes in that case will be passing through their input which is silence to their output.)
Reporter | ||
Comment 4•10 years ago
|
||
Ehsan, that sounds perfect. Thank you so much!
Assignee | ||
Comment 5•10 years ago
|
||
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8474909 [details] [diff] [review]
Add a devtools API for muting the source AudioNodes; r=roc,smaug
Again, smaug for WebIDL changes, roc for the rest.
Attachment #8474909 -
Flags: review?(roc)
Attachment #8474909 -
Flags: review?(bugs)
Attachment #8474909 -
Flags: review?(roc) → review+
Updated•10 years ago
|
Attachment #8474909 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Comment 8•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in
before you can comment on or make changes to this bug.
Description
•