Revert 7334 "adding new AEC dump start interface for chrome."
> adding new AEC dump start interface for chrome. > > This is required because we are not allow to pass CRT objects across dll boundaries, that says, when we pass a file descriptor from chrome dll to libpeerconnection dll, the file descriptor will become invalid immediate, more information can be found here: > http://msdn.microsoft.com/en-us/library/ms235460.aspx > > Chromium bug:crbug/415935 > TEST=bots > R=bjornv@webrtc.org, kwiberg@webrtc.org > > Review URL: https://webrtc-codereview.appspot.com/28629004 TBR=xians@webrtc.org Review URL: https://webrtc-codereview.appspot.com/26659004 git-svn-id: http://webrtc.googlecode.com/svn/trunk@7335 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
parent
2e417d6428
commit
875206196c
@ -12,7 +12,6 @@
|
||||
|
||||
#include <assert.h>
|
||||
|
||||
#include "webrtc/base/fileutils.h"
|
||||
#include "webrtc/common_audio/include/audio_util.h"
|
||||
#include "webrtc/common_audio/signal_processing/include/signal_processing_library.h"
|
||||
#include "webrtc/modules/audio_processing/audio_buffer.h"
|
||||
@ -717,11 +716,6 @@ int AudioProcessingImpl::StartDebugRecording(FILE* handle) {
|
||||
#endif // WEBRTC_AUDIOPROC_DEBUG_DUMP
|
||||
}
|
||||
|
||||
int AudioProcessingImpl::StartDebugRecording(rtc::PlatformFile handle) {
|
||||
FILE* stream = rtc::FdopenPlatformFileForWriting(handle);
|
||||
return StartDebugRecording(stream);
|
||||
}
|
||||
|
||||
int AudioProcessingImpl::StopDebugRecording() {
|
||||
CriticalSectionScoped crit_scoped(crit_);
|
||||
|
||||
|
@ -125,7 +125,6 @@ class AudioProcessingImpl : public AudioProcessing {
|
||||
virtual int StartDebugRecording(
|
||||
const char filename[kMaxFilenameSize]) OVERRIDE;
|
||||
virtual int StartDebugRecording(FILE* handle) OVERRIDE;
|
||||
virtual int StartDebugRecording(rtc::PlatformFile handle) OVERRIDE;
|
||||
virtual int StopDebugRecording() OVERRIDE;
|
||||
virtual EchoCancellation* echo_cancellation() const OVERRIDE;
|
||||
virtual EchoControlMobile* echo_control_mobile() const OVERRIDE;
|
||||
|
@ -14,7 +14,6 @@
|
||||
#include <stddef.h> // size_t
|
||||
#include <stdio.h> // FILE
|
||||
|
||||
#include "webrtc/base/fileutils.h"
|
||||
#include "webrtc/common.h"
|
||||
#include "webrtc/typedefs.h"
|
||||
|
||||
@ -326,11 +325,6 @@ class AudioProcessing {
|
||||
// of |handle| and closes it at StopDebugRecording().
|
||||
virtual int StartDebugRecording(FILE* handle) = 0;
|
||||
|
||||
// Same as above but uses an existing PlatformFile handle. Takes ownership
|
||||
// of |handle| and closes it at StopDebugRecording().
|
||||
// TODO(xians): Make this interface pure virtual.
|
||||
virtual int StartDebugRecording(rtc::PlatformFile handle) { return -1; }
|
||||
|
||||
// Stops recording debugging information, and closes the file. Recording
|
||||
// cannot be resumed in the same file (without overwriting it).
|
||||
virtual int StopDebugRecording() = 0;
|
||||
|
Loading…
x
Reference in New Issue
Block a user