Added option to specify a maximum file size when recording an AEC dump.

webrtc/voice_engine/include/voe_audio_processing.h

Index: webrtc/voice_engine/include/voe_audio_processing.h
diff --git a/webrtc/voice_engine/include/voe_audio_processing.h b/webrtc/voice_engine/include/voe_audio_processing.h
index fd70f957752ee1114111ac494c39ad4c2c7813a4..0c85f374d242d8dd3e3f505cdd541ce2ce384a40 100644
--- a/webrtc/voice_engine/include/voe_audio_processing.h
+++ b/webrtc/voice_engine/include/voe_audio_processing.h
@@ -190,10 +190,17 @@ class WEBRTC_DLLEXPORT VoEAudioProcessing {
   // The file can later be used for off-line analysis of the AP performance.
   virtual int StartDebugRecording(const char* fileNameUTF8) = 0;
 
+  // Same as above but with a specified maximum file size in bytes.
+  virtual int StartDebugRecording(const char* fileNameUTF8,

[Andrew MacDonald, 2015/10/23 17:15:18]

The preferred method now is to access the AudioProcessing instance directly in
libjingle, through:
voe_wrapper_->base()->audio_processing()

This wrapper layer should go away entirely with solenberg's refactoring; it
would be nicer to avoid adding new APIs, but I leave it up to you guys.

[the sun, 2015/10/26 10:37:20]

Thanks Andrew; indeed we don't want to and don't have to add the extra
interfaces here since we can get to the APM directly.

[ivoc, 2015/11/05 13:14:46]

I removed this function, should I remove the StartDebugRecording() function that
was already here as well?

+                                  int max_size_bytes) = 0;
+
   // Same as above but sets and uses an existing file handle. Takes ownership
   // of |file_handle| and passes it on to the audio processing module.
   virtual int StartDebugRecording(FILE* file_handle) = 0;
 
+  // Same as above but with a specified maximum file size in bytes.
+  virtual int StartDebugRecording(FILE* file_handle, int max_size_bytes) = 0;
+
   // Disables recording of AP debugging information.
   virtual int StopDebugRecording() = 0;