Reenable the WebRTC EventLog API Tests.

chrome/browser/extensions/api/webrtc_logging_private/webrtc_event_log_apitest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <memory>
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/json/json_writer.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 20 matching lines @@
 #endif
 
 using extensions::WebrtcLoggingPrivateStartWebRtcEventLoggingFunction;
 using extensions::WebrtcLoggingPrivateStopWebRtcEventLoggingFunction;
 
 namespace utils = extension_function_test_utils;
 
 namespace {
 
 // Get the expected EventLog file name. The name will be
 // <temporary path>.<render process id>.event_log.<consumer id>, for example

[terelius1, 2016/07/08 10:58:37]

Please update the comment.

[Ivo-OOO until feb 6, 2016/07/08 11:24:35]

Done. I renamed the variable below as well.

 // /tmp/.org.chromium.Chromium.vsygNQ/dnFW8ch/Default/WebRTC
-// Logs/WebRtcEventLog.1.29113.event_log.1
+// Logs/WebRtcEventLog.1.6.1
 base::FilePath GetExpectedEventLogFileName(const base::FilePath& base_file,
                                            int render_process_id) {
   static const int kExpectedConsumerId = 1;
   return base_file.AddExtension(IntToStringType(render_process_id))
-      .AddExtension(FILE_PATH_LITERAL("event_log"))
       .AddExtension(IntToStringType(kExpectedConsumerId));
 }
 
 static const char kMainWebrtcTestHtmlPage[] = "/webrtc/webrtc_jsep01_test.html";
 
 std::string ParamsToString(const base::ListValue& parameters) {
   std::string parameter_string;
   EXPECT_TRUE(base::JSONWriter::Write(parameters, &parameter_string));
   return parameter_string;
 }
@@ skipping 41 matching lines @@
     parameters->Append(std::move(request_info));
     parameters->AppendString(tab->GetURL().GetOrigin().spec());
   }
 
  private:
   scoped_refptr<extensions::Extension> extension_;
 };
 
 }  // namespace
 
-// TODO(ivoc): Reenable when the event log functionality in Chrome is updated.
-IN_PROC_BROWSER_TEST_F(WebrtcEventLogApiTest,
-                       DISABLED_TestStartStopWebRtcEventLogging) {
+IN_PROC_BROWSER_TEST_F(WebrtcEventLogApiTest, TestStartStopWebRtcEventLogging) {
   ASSERT_TRUE(embedded_test_server()->Start());
 
   content::WebContents* left_tab =
       OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage);
   content::WebContents* right_tab =
       OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage);
 
   SetupPeerconnectionWithLocalStream(left_tab);
   SetupPeerconnectionWithLocalStream(right_tab);
 
@@ skipping 48 matching lines @@
   ASSERT_TRUE(recordings_info_stop.get());
   base::FilePath file_name_stop(
       base::FilePath::FromUTF8Unsafe(recordings_info_stop->prefix_path));
 
   HangUp(left_tab);
   HangUp(right_tab);
 
   EXPECT_EQ(file_name_start, file_name_stop);
 
   // Check that the file exists and is non-empty.
-  base::ProcessId render_process_id =
-      base::GetProcId(left_tab->GetRenderProcessHost()->GetHandle());
-  EXPECT_NE(render_process_id, base::kNullProcessId);
+  content::RenderProcessHost* render_process_host =
+      left_tab->GetRenderProcessHost();
+  ASSERT_NE(render_process_host, nullptr);
+  int render_process_id = render_process_host->GetID();
   base::FilePath full_file_name =
       GetExpectedEventLogFileName(file_name_stop, render_process_id);
   int64_t file_size = 0;
+  int timeout_counter = 0;
   while (!(base::PathExists(full_file_name) &&
            base::GetFileSize(full_file_name, &file_size) && file_size > 0)) {
     // This should normally not happen, but is here to prevent the test
     // from becoming flaky on devices with weird timings or when the
     // /webrtc/webrtc_jsep01_test.html changes.
     VLOG(1) << "Waiting for logfile to become available...";
     base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(100));
+    if (timeout_counter++ >= 50) {
+      VLOG(1) << "Giving up on waiting for logfile.";
+      break;
+    }
   }
   ASSERT_TRUE(base::PathExists(full_file_name));
   EXPECT_TRUE(base::GetFileSize(full_file_name, &file_size));
   EXPECT_GT(file_size, 0);
 
   // Clean up.
   base::DeleteFile(full_file_name, false);
 }
 
-// TODO(ivoc): Reenable when the event log functionality in Chrome is updated.
 IN_PROC_BROWSER_TEST_F(WebrtcEventLogApiTest,
-                       DISABLED_TestStartTimedWebRtcEventLogging) {
+                       TestStartTimedWebRtcEventLogging) {
   ASSERT_TRUE(embedded_test_server()->Start());
 
   content::WebContents* left_tab =
       OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage);
   content::WebContents* right_tab =
       OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage);
 
   SetupPeerconnectionWithLocalStream(left_tab);
   SetupPeerconnectionWithLocalStream(right_tab);
 
@@ skipping 29 matching lines @@
   // Video is choppy on Mac OS X. http://crbug.com/443542.
   WaitForVideoToPlay(left_tab);
   WaitForVideoToPlay(right_tab);
 #endif
 
   HangUp(left_tab);
   HangUp(right_tab);
 
   // The log has stopped automatically. Check that the file exists and is
   // non-empty.
-  base::ProcessId render_process_id =
-      base::GetProcId(left_tab->GetRenderProcessHost()->GetHandle());
-  EXPECT_NE(render_process_id, base::kNullProcessId);
+  content::RenderProcessHost* render_process_host =
+      left_tab->GetRenderProcessHost();
+  ASSERT_NE(render_process_host, nullptr);
+  int render_process_id = render_process_host->GetID();
   base::FilePath full_file_name =
       GetExpectedEventLogFileName(file_name_start, render_process_id);
   int64_t file_size = 0;
+  int timeout_counter = 0;
   while (!(base::PathExists(full_file_name) &&
            base::GetFileSize(full_file_name, &file_size) && file_size > 0)) {
     // This should normally not happen, but is here to prevent the test
     // from becoming flaky on devices with weird timings or when the
     // /webrtc/webrtc_jsep01_test.html changes.
     VLOG(1) << "Waiting for logfile to become available...";
     base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(100));
+    if (timeout_counter++ >= 50) {
+      VLOG(1) << "Giving up on waiting for logfile.";
+      break;
+    }
   }
   ASSERT_TRUE(base::PathExists(full_file_name));
   EXPECT_TRUE(base::GetFileSize(full_file_name, &file_size));
   EXPECT_GT(file_size, 0);
 
   // Clean up.
   base::DeleteFile(full_file_name, false);
 }