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

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

Issue 2420983002: AudioParams with automations must process timelines (Closed)
Patch Set: Use kRenderQuantumFrames. Created 4 years, 1 month 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.h
diff --git a/third_party/WebKit/Source/modules/webaudio/AudioNode.h b/third_party/WebKit/Source/modules/webaudio/AudioNode.h
index 71adc2dc9b586fa158271080e7aa5482efd2d00f..ca3a043635bda6a8efe0fabb3bf5384a278aba84 100644
--- a/third_party/WebKit/Source/modules/webaudio/AudioNode.h
+++ b/third_party/WebKit/Source/modules/webaudio/AudioNode.h
@@ -141,6 +141,11 @@ class MODULES_EXPORT AudioHandler : public ThreadSafeRefCounted<AudioHandler> {
// Called from context's audio thread.
virtual void process(size_t framesToProcess) = 0;
+ // Like process(), but only causes the automations to process; the
+ // normal processing of the node is bypassed. By default, we assume
+ // no AudioParams need to be updated.
+ virtual void processAudioParams(size_t framesToProcess){};
hongchan 2016/11/07 22:17:26 Can we rename this to |processAudioParamsOnly|?
Raymond Toy 2016/11/07 23:49:24 Ok. But grammatically, I think that should be |pro
+
// No significant resources should be allocated until initialize() is called.
// Processing may not occur until a node is initialized.
virtual void initialize();

Powered by Google App Engine
This is Rietveld 408576698