commit 11a9cbfa50631f9ca3e61f3f9d993450386983c1
author skvlad <skvlad@webrtc.org> Fri Oct 07 11:53:05 2016 -0700
committer Commit bot <commit-bot@chromium.org> Fri Oct 07 18:53:15 2016 +0000
tree c41a68265363693745143b44949b724694699b0f
parent 0d348d69e6c6cf58e07e12b43a06566de580bdfc

Refactoring: move ownership of RtcEventLog from Call to PeerConnection

This CL is a pure refactoring which should not result in any functinal
changes. It moves ownership of the RtcEventLog from webrtc::Call to the
webrtc::PeerConnection object.

This is done so that we can add RtcEventLog support for ICE events -
which will require the TransportController to have a pointer to the
RtcEventLog. PeerConnection is the closest common owner of both Call and
TransportController (through WebRtcSession).

BUG=webrtc:6393

Review-Url: https://codereview.webrtc.org/2353033005
Cr-Commit-Position: refs/heads/master@{#14578}

webrtc/api/rtcstatscollector_unittest.cc

diff --git a/webrtc/api/rtcstatscollector_unittest.cc b/webrtc/api/rtcstatscollector_unittest.cc
index 7103864..0da772f 100644
--- a/webrtc/api/rtcstatscollector_unittest.cc
+++ b/webrtc/api/rtcstatscollector_unittest.cc
@@ -29,6 +29,7 @@
 #include "webrtc/base/thread_checker.h"
 #include "webrtc/base/timedelta.h"
 #include "webrtc/base/timeutils.h"
+#include "webrtc/logging/rtc_event_log/rtc_event_log.h"
 #include "webrtc/media/base/fakemediaengine.h"
 #include "webrtc/p2p/base/port.h"
 
@@ -112,14 +113,15 @@
   RTCStatsCollectorTestHelper()
       : worker_thread_(rtc::Thread::Current()),
         network_thread_(rtc::Thread::Current()),
-        channel_manager_(new cricket::ChannelManager(
-            new cricket::FakeMediaEngine(),
-            worker_thread_,
-            network_thread_)),
+        channel_manager_(
+            new cricket::ChannelManager(new cricket::FakeMediaEngine(),
+                                        worker_thread_,
+                                        network_thread_)),
         media_controller_(
             MediaControllerInterface::Create(cricket::MediaConfig(),
                                              worker_thread_,
-                                             channel_manager_.get())),
+                                             channel_manager_.get(),
+                                             &event_log_)),
         session_(media_controller_.get()),
         pc_() {
     // Default return values for mocks.
@@ -148,6 +150,7 @@
 
  private:
   rtc::ScopedFakeClock fake_clock_;
+  webrtc::RtcEventLogNullImpl event_log_;
   rtc::Thread* const worker_thread_;
   rtc::Thread* const network_thread_;
   std::unique_ptr<cricket::ChannelManager> channel_manager_;