ACM CodecOwner: Test that we reset speech encoder when enabling CNG or RED

webrtc/modules/audio_coding/main/acm2/codec_owner_unittest.cc

 /*
  *  Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 38 matching lines @@
     EXPECT_TRUE(encoded_info.redundant.empty());
     EXPECT_EQ(expected_out_length, encoded_info.encoded_bytes);
     EXPECT_EQ(expected_timestamp, encoded_info.encoded_timestamp);
     if (expected_payload_type >= 0)
       EXPECT_EQ(expected_payload_type, encoded_info.payload_type);
     if (expected_send_even_if_empty >= 0)
       EXPECT_EQ(static_cast<bool>(expected_send_even_if_empty),
                 encoded_info.send_even_if_empty);
   }
 
+  // Verify that the speech encoder's Reset method is called when CNG or RED
+  // (or both) are switched on, but not when they're switched off.
+  void TestCngAndRedResetSpeechEncoder(bool use_cng, bool use_red) {
+    MockAudioEncoder speech_encoder;
+    EXPECT_CALL(speech_encoder, NumChannels())
+        .WillRepeatedly(Return(1));
+    EXPECT_CALL(speech_encoder, Max10MsFramesInAPacket())
+        .WillRepeatedly(Return(2));
+    EXPECT_CALL(speech_encoder, SampleRateHz())
+        .WillRepeatedly(Return(8000));
+    {
+      InSequence s;
+      EXPECT_CALL(speech_encoder, Mark("start off"));
+      EXPECT_CALL(speech_encoder, Mark("switch on"));
+      if (use_cng || use_red)
+        EXPECT_CALL(speech_encoder, Reset());
+      EXPECT_CALL(speech_encoder, Mark("start on"));
+      if (use_cng || use_red)
+        EXPECT_CALL(speech_encoder, Reset());
+      EXPECT_CALL(speech_encoder, Mark("switch off"));
+      EXPECT_CALL(speech_encoder, Die());
+    }
+
+    int cng_pt = use_cng ? 17 : -1;
+    int red_pt = use_red ? 19 : -1;
+    speech_encoder.Mark("start off");
+    codec_owner_.SetEncoders(&speech_encoder, -1, VADNormal, -1);
+    speech_encoder.Mark("switch on");
+    codec_owner_.ChangeCngAndRed(cng_pt, VADNormal, red_pt);
+    speech_encoder.Mark("start on");
+    codec_owner_.SetEncoders(&speech_encoder, cng_pt, VADNormal, red_pt);
+    speech_encoder.Mark("switch off");
+    codec_owner_.ChangeCngAndRed(-1, VADNormal, -1);
+  }
+
   CodecOwner codec_owner_;
   uint32_t timestamp_;
 };
 
 // This test verifies that CNG frames are delivered as expected. Since the frame
 // size is set to 20 ms, we expect the first encode call to produce no output
 // (which is signaled as 0 bytes output of type kNoEncoding). The next encode
 // call should produce one SID frame of 9 bytes. The third call should not
 // result in any output (just like the first one). The fourth and final encode
 // call should produce an "empty frame", which is like no output, but with
@@ skipping 69 matching lines @@
                                         arraysize(encoded), encoded);
   external_encoder.Mark("B");
 
   // Change back to external encoder again.
   codec_owner_.SetEncoders(&external_encoder, -1, VADNormal, -1);
   info = codec_owner_.Encoder()->Encode(2, audio, arraysize(audio),
                                         arraysize(encoded), encoded);
   EXPECT_EQ(2u, info.encoded_timestamp);
 }
 
+TEST_F(CodecOwnerTest, CngResetsSpeechEncoder) {
+  TestCngAndRedResetSpeechEncoder(true, false);
+}
+
+TEST_F(CodecOwnerTest, RedResetsSpeechEncoder) {
+  TestCngAndRedResetSpeechEncoder(false, true);
+}
+
+TEST_F(CodecOwnerTest, CngAndRedResetsSpeechEncoder) {
+  TestCngAndRedResetSpeechEncoder(true, true);
+}
+
+TEST_F(CodecOwnerTest, NoCngAndRedNoSpeechEncoderReset) {
+  TestCngAndRedResetSpeechEncoder(false, false);
+}
+
 }  // namespace acm2
 }  // namespace webrtc