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

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

Issue 2242573002: Don't delay setting of channelInterpretation and channelCountMode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix grammar in comment. 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.cpp
diff --git a/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.cpp b/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.cpp
index f4f79b8f6c48e6f62fcc7daf9fe1c80635326145..89358182b59b4a6e413f362bf00243aba4535ac2 100644
--- a/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.cpp
+++ b/third_party/WebKit/Source/modules/webaudio/DeferredTaskHandler.cpp
@@ -179,52 +179,6 @@ void DeferredTaskHandler::processAutomaticPullNodes(size_t framesToProcess)
m_renderingAutomaticPullNodes[i]->processIfNecessary(framesToProcess);
}
-void DeferredTaskHandler::addChangedChannelCountMode(AudioHandler* node)
-{
- ASSERT(isGraphOwner());
- ASSERT(isMainThread());
- m_deferredCountModeChange.add(node);
-}
-
-void DeferredTaskHandler::removeChangedChannelCountMode(AudioHandler* node)
-{
- ASSERT(isGraphOwner());
-
- m_deferredCountModeChange.remove(node);
-}
-
-void DeferredTaskHandler::addChangedChannelInterpretation(AudioHandler* node)
-{
- ASSERT(isGraphOwner());
- ASSERT(isMainThread());
- m_deferredChannelInterpretationChange.add(node);
-}
-
-void DeferredTaskHandler::removeChangedChannelInterpretation(AudioHandler* node)
-{
- ASSERT(isGraphOwner());
-
- m_deferredChannelInterpretationChange.remove(node);
-}
-
-void DeferredTaskHandler::updateChangedChannelCountMode()
-{
- ASSERT(isGraphOwner());
-
- for (AudioHandler* node : m_deferredCountModeChange)
- node->updateChannelCountMode();
- m_deferredCountModeChange.clear();
-}
-
-void DeferredTaskHandler::updateChangedChannelInterpretation()
-{
- ASSERT(isGraphOwner());
-
- for (AudioHandler* node : m_deferredChannelInterpretationChange)
- node->updateChannelInterpretation();
- m_deferredChannelInterpretationChange.clear();
-}
-
DeferredTaskHandler::DeferredTaskHandler()
: m_automaticPullNodesNeedUpdating(false)
, m_audioThread(0)
@@ -246,8 +200,6 @@ DeferredTaskHandler::~DeferredTaskHandler()
void DeferredTaskHandler::handleDeferredTasks()
{
- updateChangedChannelCountMode();
- updateChangedChannelInterpretation();
handleDirtyAudioSummingJunctions();
handleDirtyAudioNodeOutputs();
updateAutomaticPullNodes();

Powered by Google App Engine
This is Rietveld 408576698