Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(114)

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. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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
This is Rietveld 408576698