Refactoring: move ownership of RtcEventLog from Call to PeerConnection

webrtc/call.h

Index: webrtc/call.h
diff --git a/webrtc/call.h b/webrtc/call.h
index 193124ee179879207b7218d26a6d46ee5a48cbd8..c55f6da5959e80e8b1de3ce5ebf72f115ef5ea05 100644
--- a/webrtc/call.h
+++ b/webrtc/call.h
@@ -26,6 +26,7 @@
 namespace webrtc {
 
 class AudioProcessing;
+class RtcEventLog;
 
 const char* Version();
 
@@ -89,6 +90,14 @@ class Call {
     // Audio Processing Module to be used in this call.
     // TODO(solenberg): Change this to a shared_ptr once we can use C++11.
     AudioProcessing* audio_processing = nullptr;
+
+    // RtcEventLog to use for this call. Required.
+    // Use webrtc::RtcEventLog::CreateNull() for a null implementation.
+    RtcEventLog* event_log = nullptr;
+
+    explicit Config(RtcEventLog* event_log) : event_log(event_log) {
+      RTC_DCHECK(event_log);
+    }

[stefan-webrtc, 2016/10/06 08:53:14]

Move this to the top as methods and ctors usually go before members.

[skvlad, 2016/10/07 01:35:04]

Done.

   };
 
   struct Stats {
@@ -151,10 +160,6 @@ class Call {
 
   virtual void OnSentPacket(const rtc::SentPacket& sent_packet) = 0;
 
-  virtual bool StartEventLog(rtc::PlatformFile log_file,
-                             int64_t max_size_bytes) = 0;
-  virtual void StopEventLog() = 0;
-
   virtual ~Call() {}
 };