Move the call to enable the WebRTC event log from PeerConnectionFactory to PeerConnection.

content/browser/media/webrtc/webrtc_eventlog_host.h

+// 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.
+
+#ifndef CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_EVENTLOG_HOST_H_
+#define CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_EVENTLOG_HOST_H_
+
+#include <vector>
+
+#include "base/files/file_path.h"
+#include "base/memory/weak_ptr.h"
+#include "ipc/ipc_platform_file.h"
+
+namespace content {
+
+// This class is used to enable and disable WebRTC event logs on each of the
+// PeerConnections in the render process. This class and all its members should
+// only be accessed from the Browser UI thread.
+class WebRTCEventLogHost {
+ public:
+  explicit WebRTCEventLogHost(int render_process_id);
+  ~WebRTCEventLogHost();
+
+  // Starts an RTC event log for all current and future PeerConnections on the
+  // render process. A base file_path can be supplied, which will be extended to
+  // include several identifiers to ensure uniqueness. If a recording was
+  // already in progress, this call will return false and have no other effect.
+  // Should be called from the Browser UI thread.

[Henrik Grunell, 2016/06/03 07:40:53]

Nit: Remove thread comment.

[Ivo-OOO until feb 6, 2016/06/03 16:07:02]

Done.

+  bool StartWebRTCEventLog(const base::FilePath& file_path);
+
+  // Stops recording an RTC event log for each PeerConnection on the render
+  // process. If no recording was in progress, this call will return false.
+  bool StopWebRTCEventLog();
+
+  // This function should be used to notify the WebRTCEventLogHost object that a
+  // PeerConnection was created in the corresponding render process.
+  void PeerConnectionAdded(int peer_connection_local_id);
+
+  // This function should be used to notify the WebRTCEventLogHost object that a
+  // PeerConnection was removed in the corresponding render process.
+  void PeerConnectionRemoved(int peer_connection_local_id);
+
+ private:
+  // Actually start the eventlog for a single PeerConnection using the path
+  // stored in base_file_path_.
+  bool DoStartEventLog(int peer_connection_local_id);
+
+  // Send the platform file to the render process using an IPC message.
+  void SendEventLogFileToRenderer(int peer_connection_local_id,
+                                  IPC::PlatformFileForTransit file_for_transit);
+
+  // The render process ID that this object is associated with.
+  const int render_process_id_;
+
+  // In case new PeerConnections are created during logging, the path is needed
+  // to enable logging for them.
+  base::FilePath base_file_path_;
+
+  // The local identifiers of all the currently active PeerConnections.
+  std::vector<int> active_peer_connection_local_ids_;
+
+  // Number of active log files that have been opened.
+  static int number_active_log_files_;
+
+  // Track if the RTC event log is currently active.
+  bool rtc_event_logging_enabled_;
+
+  base::WeakPtrFactory<WebRTCEventLogHost> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(WebRTCEventLogHost);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_EVENTLOG_HOST_H_