Move RtcEventLog object from inside VoiceEngine to Call.

webrtc/voice_engine/test/auto_test/standard/codec_test.cc

 /*
  *  Copyright (c) 2011 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 158 matching lines @@
   for (int i = 0; i < voe_codec_->NumOfCodecs(); ++i) {
     voe_codec_->GetCodec(i, codec_instance_);
     if (!_stricmp("opus", codec_instance_.plname)) {
       continue;
     }
     voe_codec_->SetSendCodec(channel_, codec_instance_);
     EXPECT_EQ(-1, voe_codec_->SetOpusDtx(channel_, true));
   }
 }
 
-#ifdef ENABLE_RTC_EVENT_LOG
-TEST_F(CodecTest, RtcEventLogIntegrationTest) {

[terelius, 2016/03/21 20:47:46]

Should there be an integration test for the new API too, or is it so simple that
isn't needed?

[ivoc, 2016/03/22 13:44:55]

I would like to add one, but there is currently no support for opening platform
files in WebRTC. It would be a good idea to try to add this in a follow-up CL.

-  webrtc::RtcEventLog* event_log = voe_codec_->GetEventLog();
-  ASSERT_TRUE(event_log);
-
-  // Find the name of the current test, in order to use it as a temporary
-  // filename.
-  auto test_info = ::testing::UnitTest::GetInstance()->current_test_info();
-  const std::string temp_filename = webrtc::test::OutputPath() +
-                                    test_info->test_case_name() +
-                                    test_info->name();
-  // Create a log file.
-  event_log->StartLogging(temp_filename, 1000);
-  event_log->StopLogging();
-
-  // Check if the file has been created.
-  FILE* event_file = fopen(temp_filename.c_str(), "r");
-  ASSERT_TRUE(event_file);
-  fclose(event_file);
-  // Remove the temporary file.
-  remove(temp_filename.c_str());
-}
-#endif  // ENABLE_RTC_EVENT_LOG
-
 // TODO(xians, phoglund): Re-enable when issue 372 is resolved.
 TEST_F(CodecTest, DISABLED_ManualVerifySendCodecsForAllPacketSizes) {
   for (int i = 0; i < voe_codec_->NumOfCodecs(); ++i) {
     voe_codec_->GetCodec(i, codec_instance_);
     if (IsNotViableSendCodec(codec_instance_.plname)) {
       TEST_LOG("Skipping %s.\n", codec_instance_.plname);
       continue;
     }
     EXPECT_NE(-1, codec_instance_.pltype) <<
         "The codec database should suggest a payload type.";
@@ skipping 17 matching lines @@
         TEST_LOG("%d ", packet_size);
         TEST_LOG_FLUSH;
         at_least_one_succeeded = true;
         Sleep(CODEC_TEST_TIME);
       }
     }
     TEST_LOG("\n");
     EXPECT_TRUE(at_least_one_succeeded);
   }
 }