From e56c7636912db2f859c4d1bf11e614780001bb0f Mon Sep 17 00:00:00 2001 From: minyue Date: Wed, 4 Nov 2015 07:11:48 -0800 Subject: [PATCH] Removing trace checks in VoETestManager. Trace checks in the ctor and dtor of VoETestManager are removed, since they can fail if there are more than one VoETestManager (or VoE) used in a test. BUG= Review URL: https://codereview.webrtc.org/1407883007 Cr-Commit-Position: refs/heads/master@{#10507} --- .../test/auto_test/voe_standard_test.cc | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/webrtc/voice_engine/test/auto_test/voe_standard_test.cc b/webrtc/voice_engine/test/auto_test/voe_standard_test.cc index 22eee15bf0..a187c4bb6c 100644 --- a/webrtc/voice_engine/test/auto_test/voe_standard_test.cc +++ b/webrtc/voice_engine/test/auto_test/voe_standard_test.cc @@ -117,14 +117,6 @@ bool VoETestManager::Init() { if (initialized_) return true; - if (VoiceEngine::SetTraceFile(NULL) != -1) { - // should not be possible to call a Trace method before the VoE is - // created - TEST_LOG("\nError at line: %i (VoiceEngine::SetTraceFile()" - "should fail)!\n", __LINE__); - return false; - } - voice_engine_ = VoiceEngine::Create(); if (!voice_engine_) { TEST_LOG("Failed to create VoiceEngine\n"); @@ -230,11 +222,6 @@ int VoETestManager::ReleaseInterfaces() { releaseOK = false; } - if (VoiceEngine::SetTraceFile(NULL) != -1) { - TEST_LOG("\nError at line: %i (VoiceEngine::SetTraceFile()" - "should fail)!\n", __LINE__); - } - return (releaseOK == true) ? 0 : -1; }