Changed mixing api and removed resampler

webrtc/modules/audio_mixer/test/audio_mixer_unittest.cc

 /*
  *  Copyright (c) 2016 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 88 matching lines @@
   CompareWithOldMixerTest()
       : old_mixer_(AudioConferenceMixer::Create(kId)),
         new_mixer_(NewAudioConferenceMixer::Create(kId)) {}
 
   ~CompareWithOldMixerTest() { Reset(); }
 
   // Mixes with both mixers and compares results: resulting frames and
   // mix statuses.
   void MixAndCompare() {
     old_mixer_->Process();
-    new_mixer_->Mix(&new_mixer_frame_);
+    new_mixer_->Mix(kSampleRateHz,

[aleloi, 2016/08/05 09:52:22]

Adepted tests to match new mixer API.

+                    1,  // number of channels
+                    &new_mixer_frame_);
     EXPECT_EQ(0, memcmp(old_mixer_frame_.data_, new_mixer_frame_.data_,
                         sizeof(old_mixer_frame_.data_)));
 
     for (auto& participant_pair : participants_) {
       EXPECT_EQ(participant_pair.first->IsMixed(),
                 participant_pair.second->IsMixed());
     }
   }
 
   std::unique_ptr<AudioFrame> last_mixed_audio_old() {
@@ skipping 185 matching lines @@
     EXPECT_CALL(participants[i], NeededFrequency(_))
         .WillRepeatedly(Return(kSampleRateHz));
   }
 
   // Last participant gives audio frame with passive VAD, although it has the
   // largest energy.
   participants[kAudioSources - 1].fake_frame()->vad_activity_ =
       AudioFrame::kVadPassive;
 
   AudioFrame audio_frame;
-  mixer->Mix(&audio_frame);
+  mixer->Mix(kSampleRateHz,
+             1,  // number of channels
+             &audio_frame);
 
   for (int i = 0; i < kAudioSources; ++i) {
     bool is_mixed = participants[i].IsMixed();
     if (i == kAudioSources - 1 ||
         i < kAudioSources - 1 -
                 NewAudioConferenceMixer::kMaximumAmountOfMixedAudioSources) {
       EXPECT_FALSE(is_mixed) << "Mixing status of AudioSource #" << i
                              << " wrong.";
     } else {
       EXPECT_TRUE(is_mixed) << "Mixing status of AudioSource #" << i
                             << " wrong.";
     }
   }
 }
 
 TEST_F(BothMixersTest, CompareInitialFrameAudio) {
   EXPECT_CALL(participant_, GetAudioFrameWithMuted(_, _)).Times(Exactly(1));
 
   // Make sure the participant is marked as 'non-mixed' so that it is
   // ramped in next round.
   ResetAudioSource();
 
   // Construct the expected sound for the first mixing round.
   mixing_round_frame.CopyFrom(*participant_.fake_frame());
   RampIn(mixing_round_frame);
 
   // Mix frames and put the result into a frame.
-  audio_mixer_->MixActiveChannels();

[aleloi, 2016/08/05 09:52:22]

This is the test that touches OutputMixer.

Now GetMixedAudio also causes the mixer to mix one iteration. It's a bad name
for the function, but I think it's OK, as AudioMixer (formerly OutputMixer) will
be removed anyway.

   audio_mixer_->GetMixedAudio(kSampleRateHz, 1, &mixed_results_frame_);
 
   // Compare the received frame with the expected.
   EXPECT_EQ(mixing_round_frame.sample_rate_hz_,
             mixed_results_frame_.sample_rate_hz_);
   EXPECT_EQ(mixing_round_frame.num_channels_,
             mixed_results_frame_.num_channels_);
   EXPECT_EQ(mixing_round_frame.samples_per_channel_,
             mixed_results_frame_.samples_per_channel_);
   EXPECT_EQ(0, memcmp(mixing_round_frame.data_, mixed_results_frame_.data_,
                       sizeof(mixing_round_frame.data_)));
 }
 
 TEST_F(BothMixersTest, CompareSecondFrameAudio) {
   EXPECT_CALL(participant_, GetAudioFrameWithMuted(_, _)).Times(Exactly(2));
 
   // Make sure the participant is marked as 'non-mixed' so that it is
   // ramped in next round.
   ResetAudioSource();
 
   // Do one mixing iteration.
-  audio_mixer_->MixActiveChannels();
+  audio_mixer_->GetMixedAudio(kSampleRateHz, 1, &mixed_results_frame_);
 
   // Mix frames a second time and compare with the expected frame
   // (which is the participant's frame).
-  audio_mixer_->MixActiveChannels();
   audio_mixer_->GetMixedAudio(kSampleRateHz, 1, &mixed_results_frame_);
   EXPECT_EQ(0,
             memcmp(participant_.fake_frame()->data_, mixed_results_frame_.data_,
                    sizeof(mixing_round_frame.data_)));
 }
 
 TEST_F(CompareWithOldMixerTest, TwoParticipantsNormalFrames) {
   Reset();
   AudioFrame first_frame, second_frame;
 
@@ skipping 54 matching lines @@
                      MixerParticipant::AudioFrameInfo::kMuted);
     } else {
       AddParticipant(&audio_frames[i],
                      MixerParticipant::AudioFrameInfo::kNormal);
     }
     MixAndCompare();
   }
 }
 
 }  // namespace webrtc