AecDump interface

webrtc/modules/audio_processing/include/mock_audio_processing.h

Index: webrtc/modules/audio_processing/include/mock_audio_processing.h
diff --git a/webrtc/modules/audio_processing/include/mock_audio_processing.h b/webrtc/modules/audio_processing/include/mock_audio_processing.h
index 4bda0bd090f213f05358ab2c91d2c079aff5f562..793266094723f86f49471f68cd58c45621e463c2 100644
--- a/webrtc/modules/audio_processing/include/mock_audio_processing.h
+++ b/webrtc/modules/audio_processing/include/mock_audio_processing.h
@@ -13,6 +13,7 @@
 
 #include <memory>
 
+#include "webrtc/modules/audio_processing/include/aec_dump.h"
 #include "webrtc/modules/audio_processing/include/audio_processing.h"
 #include "webrtc/test/gmock.h"
 
@@ -208,6 +209,8 @@ class MockAudioProcessing : public AudioProcessing {
 
   MOCK_CONST_METHOD0(GetConfig, AudioProcessing::Config());
 
+  virtual void StartDebugRecording(std::unique_ptr<AecDump> aec_dump) {}

[peah-webrtc, 2017/04/07 12:57:16]

Why don't we want to mock this method similarly to the other StartDebugRecording
methods?

[aleloi, 2017/04/12 11:05:30]

Mocked methods can't have move-only arguments. If we want to check that
StartDebugRecording is called in a unit test, it's possible to add a mocked
method like this:

MOCK_METHOD1(StartDebugRecordingMock, void(AecDump* dump));

and call it in the non-mocked StartDebugRecording.

+
  private:
   std::unique_ptr<MockEchoCancellation> echo_cancellation_;
   std::unique_ptr<MockEchoControlMobile> echo_control_mobile_;