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

Unified Diff: Source/modules/webaudio/AudioContext.cpp

Issue 210713005: Make WebAudio output as stereo as default (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 9 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/webaudio/AudioContext.cpp
diff --git a/Source/modules/webaudio/AudioContext.cpp b/Source/modules/webaudio/AudioContext.cpp
index 1fa9d78d1487be98833b4ca9eed55da21e13c2ac..9d6fd96748e493797f6ab58a3aad0a0cff7d88fc 100644
--- a/Source/modules/webaudio/AudioContext.cpp
+++ b/Source/modules/webaudio/AudioContext.cpp
@@ -419,8 +419,7 @@ PassRefPtr<MediaStreamAudioSourceNode> AudioContext::createMediaStreamSource(Med
PassRefPtr<MediaStreamAudioDestinationNode> AudioContext::createMediaStreamDestination()
{
// FIXME: Add support for an optional argument which specifies the number of channels.
Tommy Widenflycht 2014/03/25 09:49:27 I guess this comment can be removed as well
no longer working on chromium 2014/03/25 10:08:29 Done with replacing the FIXME with a comment, as h
- // FIXME: The default should probably be stereo instead of mono.
- return MediaStreamAudioDestinationNode::create(this, 1);
+ return MediaStreamAudioDestinationNode::create(this, 2);
Tommy Widenflycht 2014/03/25 09:49:27 Can this be made into an enum? MediaStreamAudioDes
no longer working on chromium 2014/03/25 10:08:29 :) Looking at the whole file, they use 2 for the c
}
PassRefPtr<ScriptProcessorNode> AudioContext::createScriptProcessor(ExceptionState& exceptionState)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698