Remove VoEHardware interface usage.

webrtc/media/engine/webrtcvoiceengine_unittest.cc

Index: webrtc/media/engine/webrtcvoiceengine_unittest.cc
diff --git a/webrtc/media/engine/webrtcvoiceengine_unittest.cc b/webrtc/media/engine/webrtcvoiceengine_unittest.cc
index 4c8599008ab49dbef6ca365ad461b0759357ce90..f7cfa7f3f075bfb30561065cba058e072994bf50 100644
--- a/webrtc/media/engine/webrtcvoiceengine_unittest.cc
+++ b/webrtc/media/engine/webrtcvoiceengine_unittest.cc
@@ -62,8 +62,7 @@ class FakeVoEWrapper : public cricket::VoEWrapper {
  public:
   explicit FakeVoEWrapper(cricket::FakeWebRtcVoiceEngine* engine)
       : cricket::VoEWrapper(engine,  // base
-                            engine,  // codec
-                            engine) {  // hw
+                            engine) {  // codec
   }
 };
 
@@ -74,17 +73,50 @@ class MockTransmitMixer : public webrtc::voe::TransmitMixer {
 
   MOCK_METHOD1(EnableStereoChannelSwapping, void(bool enable));
 };
+
+void AdmSetupExpectations(webrtc::test::MockAudioDeviceModule* adm) {

[henrika_webrtc, 2017/03/13 11:15:55]

These changes does not correlate 100% with the removal of the VoEHardware. Care
to elaborate on how you came up with the new (extended) expectations?

[the sun, 2017/03/15 12:47:57]

Certainly!

The previous calls to SetRecordingDevice(), SetPlayoutDevice() never touched the
ADM - they went through VoEHardware, which was provided by
FakeWebRtcVoiceEngine. Now they go directly to the ADM.

[henrika_webrtc, 2017/03/15 12:50:19]

Acknowledged.

+  RTC_DCHECK(adm);
+  EXPECT_CALL(*adm, AddRef()).WillOnce(Return(0));
+  EXPECT_CALL(*adm, Release()).WillOnce(Return(0));
+#if !defined(WEBRTC_IOS)
+  EXPECT_CALL(*adm, Recording()).WillOnce(Return(false));
+  EXPECT_CALL(*adm, SetRecordingChannel(webrtc::AudioDeviceModule::
+      ChannelType::kChannelBoth)).WillOnce(Return(0));
+#if defined(WEBRTC_WIN)
+  EXPECT_CALL(*adm, SetRecordingDevice(
+      static_cast<webrtc::AudioDeviceModule::WindowsDeviceType>(
+          webrtc::AudioDeviceModule::WindowsDeviceType::kDefaultCommunicationDevice)))
+              .WillOnce(Return(0));
+#else
+  EXPECT_CALL(*adm, SetRecordingDevice(0)).WillOnce(Return(0));
+#endif  // #if defined(WEBRTC_WIN)
+  EXPECT_CALL(*adm, InitMicrophone()).WillOnce(Return(0));
+  EXPECT_CALL(*adm, StereoRecordingIsAvailable(testing::_)).WillOnce(Return(0));
+  EXPECT_CALL(*adm, SetStereoRecording(false)).WillOnce(Return(0));
+  EXPECT_CALL(*adm, Playing()).WillOnce(Return(false));
+#if defined(WEBRTC_WIN)
+  EXPECT_CALL(*adm, SetPlayoutDevice(
+      static_cast<webrtc::AudioDeviceModule::WindowsDeviceType>(
+          webrtc::AudioDeviceModule::WindowsDeviceType::kDefaultCommunicationDevice)))
+              .WillOnce(Return(0));
+#else
+  EXPECT_CALL(*adm, SetPlayoutDevice(0)).WillOnce(Return(0));
+#endif  // #if defined(WEBRTC_WIN)
+  EXPECT_CALL(*adm, InitSpeaker()).WillOnce(Return(0));
+  EXPECT_CALL(*adm, StereoPlayoutIsAvailable(testing::_)).WillOnce(Return(0));
+  EXPECT_CALL(*adm, SetStereoPlayout(false)).WillOnce(Return(0));
+#endif  // #if !defined(WEBRTC_IOS)
+  EXPECT_CALL(*adm, BuiltInAECIsAvailable()).WillOnce(Return(false));
+  EXPECT_CALL(*adm, BuiltInAGCIsAvailable()).WillOnce(Return(false));
+  EXPECT_CALL(*adm, BuiltInNSIsAvailable()).WillOnce(Return(false));
+  EXPECT_CALL(*adm, SetAGC(true)).WillOnce(Return(0));
+}
 }  // namespace
 
 // Tests that our stub library "works".
 TEST(WebRtcVoiceEngineTestStubLibrary, StartupShutdown) {
   StrictMock<webrtc::test::MockAudioDeviceModule> adm;
-  EXPECT_CALL(adm, AddRef()).WillOnce(Return(0));
-  EXPECT_CALL(adm, Release()).WillOnce(Return(0));
-  EXPECT_CALL(adm, BuiltInAECIsAvailable()).WillOnce(Return(false));
-  EXPECT_CALL(adm, BuiltInAGCIsAvailable()).WillOnce(Return(false));
-  EXPECT_CALL(adm, BuiltInNSIsAvailable()).WillOnce(Return(false));
-  EXPECT_CALL(adm, SetAGC(true)).WillOnce(Return(0));
+  AdmSetupExpectations(&adm);
   StrictMock<webrtc::test::MockAudioProcessing> apm;
   EXPECT_CALL(apm, ApplyConfig(testing::_));
   EXPECT_CALL(apm, SetExtraOptions(testing::_));
@@ -121,12 +153,7 @@ class WebRtcVoiceEngineTestFake : public testing::Test {
         call_(webrtc::Call::Config(&event_log_)), voe_(&apm_, &transmit_mixer_),
         override_field_trials_(field_trials) {
     // AudioDeviceModule.
-    EXPECT_CALL(adm_, AddRef()).WillOnce(Return(0));
-    EXPECT_CALL(adm_, Release()).WillOnce(Return(0));
-    EXPECT_CALL(adm_, BuiltInAECIsAvailable()).WillOnce(Return(false));
-    EXPECT_CALL(adm_, BuiltInAGCIsAvailable()).WillOnce(Return(false));
-    EXPECT_CALL(adm_, BuiltInNSIsAvailable()).WillOnce(Return(false));
-    EXPECT_CALL(adm_, SetAGC(true)).WillOnce(Return(0));
+    AdmSetupExpectations(&adm_);
     // AudioProcessing.
     EXPECT_CALL(apm_, ApplyConfig(testing::_));
     EXPECT_CALL(apm_, SetExtraOptions(testing::_));