Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(40)

Unified Diff: webrtc/modules/audio_device/test/func_test_manager.cc

Issue 2054373002: FileWrapper[Impl] modifications and actually remove the "Impl" class. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fix compile error in func_test_manager.cc Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/audio_device/test/func_test_manager.cc
diff --git a/webrtc/modules/audio_device/test/func_test_manager.cc b/webrtc/modules/audio_device/test/func_test_manager.cc
index f16f296011bcfa2bebed4bce805530848ab1d9e0..89afa1ebe085e4279fd38b5258e4db4ceadc82e8 100644
--- a/webrtc/modules/audio_device/test/func_test_manager.cc
+++ b/webrtc/modules/audio_device/test/func_test_manager.cc
@@ -169,15 +169,10 @@ int32_t AudioTransportImpl::SetFilePlayout(bool enable, const char* fileName)
{
_playFromFile = enable;
if (enable)
- {
- return (_playFile.OpenFile(fileName, true, true, false));
- } else
- {
- _playFile.Flush();
- return (_playFile.CloseFile());
- }
+ return (_playFile.OpenFile(fileName, true));
+ _playFile.CloseFile();
+ return 0;
}
-;
void AudioTransportImpl::SetFullDuplex(bool enable)
{
@@ -462,7 +457,7 @@ int32_t AudioTransportImpl::NeedMorePlayData(
}
} // if (_fullDuplex)
- if (_playFromFile && _playFile.Open())
+ if (_playFromFile && _playFile.is_open())
{
int16_t fileBuf[480];
@@ -491,7 +486,7 @@ int32_t AudioTransportImpl::NeedMorePlayData(
audio16++;
}
}
- } // if (_playFromFile && _playFile.Open())
+ } // if (_playFromFile && _playFile.is_open())
_playCount++;

Powered by Google App Engine
This is Rietveld 408576698