Chromium Code Reviews

Unified Diff: third_party/WebKit/Source/modules/webaudio/AudioNode.cpp

Issue 2420983002: AudioParams with automations must process timelines (Closed)
Patch Set: Fix paths. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: third_party/WebKit/Source/modules/webaudio/AudioNode.cpp
diff --git a/third_party/WebKit/Source/modules/webaudio/AudioNode.cpp b/third_party/WebKit/Source/modules/webaudio/AudioNode.cpp
index 04d191501478fd5121e18cff47e8dadbe78db74e..f13bc611b0263f77de1f576802b373251ca9948e 100644
--- a/third_party/WebKit/Source/modules/webaudio/AudioNode.cpp
+++ b/third_party/WebKit/Source/modules/webaudio/AudioNode.cpp
@@ -330,6 +330,10 @@ void AudioHandler::processIfNecessary(size_t framesToProcess) {
if (silentInputs && propagatesSilence()) {
silenceOutputs();
+ // AudioParams still need to be processed so that the value can be updated
+ // if there are automations or so that the upstream nodes get pulled if
+ // any are connected to the AudioParam.
+ processOnlyAudioParams(framesToProcess);
} else {
// Unsilence the outputs first because the processing of the node may
// cause the outputs to go silent and we want to propagate that hint to

Powered by Google App Engine