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

Unified Diff: media/base/channel_layout.cc

Issue 11150034: Add support for channel transforms. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 8 years, 2 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 | media/base/channel_mixer.h » ('j') | media/base/channel_mixer.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/channel_layout.cc
diff --git a/media/base/channel_layout.cc b/media/base/channel_layout.cc
index b8b55001e3191f5269cbdd405540c2bd5951b7b5..a95ac002f87447cd011161ef12d485c23653877e 100644
--- a/media/base/channel_layout.cc
+++ b/media/base/channel_layout.cc
@@ -86,7 +86,7 @@ static const int kChannelOrderings[CHANNEL_LAYOUT_MAX][CHANNELS_MAX] = {
{ 0 , 1 , 2 , 3 , 4 , 5 , 6 , 7 , -1 , -1 , -1 , -1 , -1 },
// CHANNEL_LAYOUT_STEREO_DOWNMIX
- { -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , 0 , 1 },
+ { 0 , 1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 , -1 },
scherkus (not reviewing) 2012/10/18 01:30:39 is this a bug fix that didn't happen when DOWNMIX
DaleCurtis 2012/10/18 01:56:44 I'm not sure what those channels actually mean and
scherkus (not reviewing) 2012/10/18 02:24:03 No clue. You'll have to dig through commit logs +
DaleCurtis 2012/10/18 05:22:04 Removed. History shows they were just added becaus
// FL | FR | FC | LFE | BL | BR | FLofC | FRofC | BC | SL | SR | StL | StR
};
« no previous file with comments | « no previous file | media/base/channel_mixer.h » ('j') | media/base/channel_mixer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698