Bug 1296531 - Fix AudioTrackEncoder resampling gtest to match comment. r?jesup draft
authorAndreas Pehrson <pehrsons@gmail.com>
Fri, 17 Feb 2017 12:23:43 +0100
changeset 670309 4c9cb4116f5038383767ee82aedb818f67db8a58
parent 670308 420c79e05e860eea28244d78d31bfc059afe53ec
child 670310 c5ef2cd5cb6e60f72bb459d3da6ee9d824d0dc4b
push id81598
push userbmo:apehrson@mozilla.com
push dateTue, 26 Sep 2017 09:13:19 +0000
reviewersjesup
bugs1296531
milestone58.0a1
Bug 1296531 - Fix AudioTrackEncoder resampling gtest to match comment. r?jesup MozReview-Commit-ID: F0uCA9VkeQf
dom/media/gtest/TestAudioTrackEncoder.cpp
--- a/dom/media/gtest/TestAudioTrackEncoder.cpp
+++ b/dom/media/gtest/TestAudioTrackEncoder.cpp
@@ -197,18 +197,18 @@ TEST(OpusAudioTrackEncoder, Resample)
   // (kHz)) at initialization.
   EXPECT_TRUE(TestOpusResampler(1, 8000) == 8000);
   EXPECT_TRUE(TestOpusResampler(1, 12000) == 12000);
   EXPECT_TRUE(TestOpusResampler(1, 16000) == 16000);
   EXPECT_TRUE(TestOpusResampler(1, 24000) == 24000);
   EXPECT_TRUE(TestOpusResampler(1, 48000) == 48000);
 
   // Otherwise, it should be resampled to 48kHz by resampler.
-  EXPECT_FALSE(TestOpusResampler(1, 9600) == 9600);
-  EXPECT_FALSE(TestOpusResampler(1, 44100) == 44100);
+  EXPECT_TRUE(TestOpusResampler(1, 9600) == 48000);
+  EXPECT_TRUE(TestOpusResampler(1, 44100) == 48000);
 }
 
 TEST(OpusAudioTrackEncoder, FetchMetadata)
 {
   const int32_t channels = 1;
   const int32_t sampleRate = 44100;
   TestOpusTrackEncoder encoder;
   EXPECT_TRUE(encoder.TestOpusCreation(channels, sampleRate));