X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Fmedia%2Fbase%2Fchannel_mixer_unittest.cc;h=07d64eb0359865265e05712bbce10eae1d0ddb7f;hb=1afa4dd80ef85af7c90efaea6959db1d92330844;hp=ac228e661dd39bb5869b4e98ddafa624807cdc4a;hpb=90762837333c13ccf56f2ad88e4481fc71e8d281;p=platform%2Fframework%2Fweb%2Fcrosswalk.git

diff --git a/src/media/base/channel_mixer_unittest.cc b/src/media/base/channel_mixer_unittest.cc
index ac228e6..07d64eb 100644
--- a/src/media/base/channel_mixer_unittest.cc
+++ b/src/media/base/channel_mixer_unittest.cc
@@ -24,14 +24,16 @@ TEST(ChannelMixerTest, ConstructAllPossibleLayouts) {
        input_layout <= CHANNEL_LAYOUT_MAX;
        input_layout = static_cast<ChannelLayout>(input_layout + 1)) {
     for (ChannelLayout output_layout = CHANNEL_LAYOUT_MONO;
-         output_layout < CHANNEL_LAYOUT_STEREO_DOWNMIX;
+         output_layout <= CHANNEL_LAYOUT_MAX;
          output_layout = static_cast<ChannelLayout>(output_layout + 1)) {
       // DISCRETE can't be tested here based on the current approach.
       // CHANNEL_LAYOUT_STEREO_AND_KEYBOARD_MIC is not mixable.
+      // Stereo down mix should never be the output layout.
       if (input_layout == CHANNEL_LAYOUT_DISCRETE ||
           input_layout == CHANNEL_LAYOUT_STEREO_AND_KEYBOARD_MIC ||
           output_layout == CHANNEL_LAYOUT_DISCRETE ||
-          output_layout == CHANNEL_LAYOUT_STEREO_AND_KEYBOARD_MIC) {
+          output_layout == CHANNEL_LAYOUT_STEREO_AND_KEYBOARD_MIC ||
+          output_layout == CHANNEL_LAYOUT_STEREO_DOWNMIX) {
         continue;
       }
 
@@ -141,7 +143,7 @@ TEST_P(ChannelMixerTest, Mixing) {
   if (input_layout != CHANNEL_LAYOUT_DISCRETE) {
     for (int ch = 0; ch < output_bus->channels(); ++ch) {
       for (int frame = 0; frame < output_bus->frames(); ++frame) {
-        ASSERT_FLOAT_EQ(output_bus->channel(ch)[frame], expected_value);
+        ASSERT_FLOAT_EQ(expected_value, output_bus->channel(ch)[frame]);
       }
     }
   } else {
@@ -151,7 +153,7 @@ TEST_P(ChannelMixerTest, Mixing) {
     for (int ch = 0; ch < output_bus->channels(); ++ch) {
       expected_value = (ch < input_channels) ? channel_values[ch] : 0;
       for (int frame = 0; frame < output_bus->frames(); ++frame) {
-        ASSERT_FLOAT_EQ(output_bus->channel(ch)[frame], expected_value);
+        ASSERT_FLOAT_EQ(expected_value, output_bus->channel(ch)[frame]);
       }
     }
   }