Chromium Code Reviews| Index: talk/app/webrtc/statscollector_unittest.cc |
| diff --git a/talk/app/webrtc/statscollector_unittest.cc b/talk/app/webrtc/statscollector_unittest.cc |
| index 5e658680be50f965cb46e9135efb3de2ba1dc9d5..e908687b4440114fdfd6c8e819cf571e482a9594 100644 |
| --- a/talk/app/webrtc/statscollector_unittest.cc |
| +++ b/talk/app/webrtc/statscollector_unittest.cc |
| @@ -804,8 +804,8 @@ TEST_F(StatsCollectorTest, BytesCounterHandles64Bits) { |
| Return(true))); |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, kVideoChannelName, false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, kVideoChannelName, false); |
| StatsReports reports; // returned values. |
| cricket::VideoSenderInfo video_sender_info; |
| cricket::VideoMediaInfo stats_read; |
| @@ -850,8 +850,8 @@ TEST_F(StatsCollectorTest, BandwidthEstimationInfoIsReported) { |
| Return(true))); |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, kVideoChannelName, false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, kVideoChannelName, false); |
| StatsReports reports; // returned values. |
| cricket::VideoSenderInfo video_sender_info; |
| @@ -929,8 +929,8 @@ TEST_F(StatsCollectorTest, TrackObjectExistsWithoutUpdateStats) { |
| StatsCollectorForTest stats(&session_); |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, "video", false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, "video", false); |
| AddOutgoingVideoTrackStats(); |
| stats.AddStream(stream_); |
| @@ -965,8 +965,8 @@ TEST_F(StatsCollectorTest, TrackAndSsrcObjectExistAfterUpdateSsrcStats) { |
| Return(true))); |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, kVideoChannelName, false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, kVideoChannelName, false); |
| AddOutgoingVideoTrackStats(); |
| stats.AddStream(stream_); |
| @@ -1029,8 +1029,8 @@ TEST_F(StatsCollectorTest, TransportObjectLinkedFromSsrcObject) { |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| // The transport_name known by the video channel. |
| const std::string kVcName("vcname"); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, kVcName, false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, kVcName, false); |
| AddOutgoingVideoTrackStats(); |
| stats.AddStream(stream_); |
| @@ -1087,8 +1087,8 @@ TEST_F(StatsCollectorTest, RemoteSsrcInfoIsAbsent) { |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| // The transport_name known by the video channel. |
| const std::string kVcName("vcname"); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, kVcName, false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, kVcName, false); |
| AddOutgoingVideoTrackStats(); |
| stats.AddStream(stream_); |
| @@ -1116,8 +1116,8 @@ TEST_F(StatsCollectorTest, RemoteSsrcInfoIsPresent) { |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| // The transport_name known by the video channel. |
| const std::string kVcName("vcname"); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, kVcName, false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, kVcName, false); |
| AddOutgoingVideoTrackStats(); |
| stats.AddStream(stream_); |
| @@ -1171,8 +1171,8 @@ TEST_F(StatsCollectorTest, ReportsFromRemoteTrack) { |
| Return(true))); |
| MockVideoMediaChannel* media_channel = new MockVideoMediaChannel(); |
| - cricket::VideoChannel video_channel(rtc::Thread::Current(), |
| - media_channel, NULL, kVideoChannelName, false); |
| + cricket::VideoChannel video_channel(rtc::Thread::Current(), media_channel, |
| + nullptr, NULL, kVideoChannelName, false); |
| AddIncomingVideoTrackStats(); |
| stats.AddStream(stream_); |
| @@ -1486,8 +1486,9 @@ TEST_F(StatsCollectorTest, GetStatsFromLocalAudioTrack) { |
| MockVoiceMediaChannel* media_channel = new MockVoiceMediaChannel(); |
| // The transport_name known by the voice channel. |
| const std::string kVcName("vcname"); |
| - cricket::VoiceChannel voice_channel(rtc::Thread::Current(), |
| - media_engine_, media_channel, NULL, kVcName, false); |
| + cricket::VoiceChannel voice_channel(rtc::Thread::Current(), media_engine_, |
| + media_channel, nullptr, NULL, kVcName, |
| + false); |
|
pthatcher1
2015/10/05 18:30:13
Can you make all the "nullptr, NULL" instances be
stefan-webrtc
2015/10/07 16:55:24
Done.
|
| AddOutgoingAudioTrackStats(); |
| stats.AddStream(stream_); |
| stats.AddLocalAudioTrack(audio_track_, kSsrcOfTrack); |
| @@ -1521,8 +1522,9 @@ TEST_F(StatsCollectorTest, GetStatsFromRemoteStream) { |
| MockVoiceMediaChannel* media_channel = new MockVoiceMediaChannel(); |
| // The transport_name known by the voice channel. |
| const std::string kVcName("vcname"); |
| - cricket::VoiceChannel voice_channel(rtc::Thread::Current(), |
| - media_engine_, media_channel, NULL, kVcName, false); |
| + cricket::VoiceChannel voice_channel(rtc::Thread::Current(), media_engine_, |
| + media_channel, nullptr, NULL, kVcName, |
| + false); |
| AddIncomingAudioTrackStats(); |
| stats.AddStream(stream_); |
| @@ -1550,8 +1552,9 @@ TEST_F(StatsCollectorTest, GetStatsAfterRemoveAudioStream) { |
| MockVoiceMediaChannel* media_channel = new MockVoiceMediaChannel(); |
| // The transport_name known by the voice channel. |
| const std::string kVcName("vcname"); |
| - cricket::VoiceChannel voice_channel(rtc::Thread::Current(), |
| - media_engine_, media_channel, NULL, kVcName, false); |
| + cricket::VoiceChannel voice_channel(rtc::Thread::Current(), media_engine_, |
| + media_channel, nullptr, NULL, kVcName, |
| + false); |
| AddOutgoingAudioTrackStats(); |
| stats.AddStream(stream_); |
| stats.AddLocalAudioTrack(audio_track_.get(), kSsrcOfTrack); |
| @@ -1611,8 +1614,9 @@ TEST_F(StatsCollectorTest, LocalAndRemoteTracksWithSameSsrc) { |
| MockVoiceMediaChannel* media_channel = new MockVoiceMediaChannel(); |
| // The transport_name known by the voice channel. |
| const std::string kVcName("vcname"); |
| - cricket::VoiceChannel voice_channel(rtc::Thread::Current(), |
| - media_engine_, media_channel, NULL, kVcName, false); |
| + cricket::VoiceChannel voice_channel(rtc::Thread::Current(), media_engine_, |
| + media_channel, nullptr, NULL, kVcName, |
| + false); |
| // Create a local stream with a local audio track and adds it to the stats. |
| AddOutgoingAudioTrackStats(); |
| @@ -1698,8 +1702,9 @@ TEST_F(StatsCollectorTest, TwoLocalTracksWithSameSsrc) { |
| MockVoiceMediaChannel* media_channel = new MockVoiceMediaChannel(); |
| // The transport_name known by the voice channel. |
| const std::string kVcName("vcname"); |
| - cricket::VoiceChannel voice_channel(rtc::Thread::Current(), |
| - media_engine_, media_channel, NULL, kVcName, false); |
| + cricket::VoiceChannel voice_channel(rtc::Thread::Current(), media_engine_, |
| + media_channel, nullptr, NULL, kVcName, |
| + false); |
| // Create a local stream with a local audio track and adds it to the stats. |
| AddOutgoingAudioTrackStats(); |