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

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

Issue 2276973002: Revert CL 2242573002 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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/DeferredTaskHandler.h
diff --git a/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.h b/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.h
index 857e088c1869fe7b569889d0acdd13ddb13ef6a8..afd32cce61f69cb488854e616abac03fc702dfd4 100644
--- a/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.h
+++ b/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.h
@@ -161,6 +161,8 @@ public:
private:
DeferredTaskHandler();
void updateAutomaticPullNodes();
+ void updateChangedChannelCountMode();
+ void updateChangedChannelInterpretation();
void handleDirtyAudioSummingJunctions();
void handleDirtyAudioNodeOutputs();
void deleteHandlersOnMainThread();
@@ -174,6 +176,13 @@ private:
// m_automaticPullNodesNeedUpdating keeps track if m_automaticPullNodes is modified.
bool m_automaticPullNodesNeedUpdating;
+ // Collection of nodes where the channel count mode has changed. We want the
+ // channel count mode to change in the pre- or post-rendering phase so as
+ // not to disturb the running audio thread.
+ HashSet<AudioHandler*> m_deferredCountModeChange;
+
+ HashSet<AudioHandler*> m_deferredChannelInterpretationChange;
+
// These two HashSet must be accessed only when the graph lock is held.
// These raw pointers are safe because their destructors unregister them.
HashSet<AudioSummingJunction*> m_dirtySummingJunctions;

Powered by Google App Engine
This is Rietveld 408576698