Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(88)

Side by Side Diff: media/cast/rtcp/rtcp_receiver_unittest.cc

Issue 210303003: Cast: Remove LoggingStats in favor of event subscribers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix compile Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/memory/scoped_ptr.h" 5 #include "base/memory/scoped_ptr.h"
6 #include "base/test/simple_test_tick_clock.h" 6 #include "base/test/simple_test_tick_clock.h"
7 #include "media/cast/cast_environment.h" 7 #include "media/cast/cast_environment.h"
8 #include "media/cast/rtcp/mock_rtcp_receiver_feedback.h" 8 #include "media/cast/rtcp/mock_rtcp_receiver_feedback.h"
9 #include "media/cast/rtcp/mock_rtcp_sender_feedback.h" 9 #include "media/cast/rtcp/mock_rtcp_sender_feedback.h"
10 #include "media/cast/rtcp/rtcp_receiver.h" 10 #include "media/cast/rtcp/rtcp_receiver.h"
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 155
156 class RtcpReceiverTest : public ::testing::Test { 156 class RtcpReceiverTest : public ::testing::Test {
157 protected: 157 protected:
158 RtcpReceiverTest() 158 RtcpReceiverTest()
159 : testing_clock_(new base::SimpleTestTickClock()), 159 : testing_clock_(new base::SimpleTestTickClock()),
160 task_runner_(new test::FakeSingleThreadTaskRunner(testing_clock_)), 160 task_runner_(new test::FakeSingleThreadTaskRunner(testing_clock_)),
161 cast_environment_(new CastEnvironment( 161 cast_environment_(new CastEnvironment(
162 scoped_ptr<base::TickClock>(testing_clock_).Pass(), 162 scoped_ptr<base::TickClock>(testing_clock_).Pass(),
163 task_runner_, 163 task_runner_,
164 task_runner_, 164 task_runner_,
165 task_runner_, 165 task_runner_)),
166 GetDefaultCastReceiverLoggingConfig())),
167 rtcp_receiver_(new RtcpReceiver(cast_environment_, 166 rtcp_receiver_(new RtcpReceiver(cast_environment_,
168 &mock_sender_feedback_, 167 &mock_sender_feedback_,
169 &mock_receiver_feedback_, 168 &mock_receiver_feedback_,
170 &mock_rtt_feedback_, 169 &mock_rtt_feedback_,
171 kSourceSsrc)) { 170 kSourceSsrc)) {
172 EXPECT_CALL(mock_receiver_feedback_, OnReceivedSenderReport(_)).Times(0); 171 EXPECT_CALL(mock_receiver_feedback_, OnReceivedSenderReport(_)).Times(0);
173 EXPECT_CALL(mock_receiver_feedback_, OnReceiverReferenceTimeReport(_)) 172 EXPECT_CALL(mock_receiver_feedback_, OnReceiverReferenceTimeReport(_))
174 .Times(0); 173 .Times(0);
175 EXPECT_CALL(mock_receiver_feedback_, OnReceivedSendReportRequest()) 174 EXPECT_CALL(mock_receiver_feedback_, OnReceivedSendReportRequest())
176 .Times(0); 175 .Times(0);
(...skipping 406 matching lines...) Expand 10 before | Expand all | Expand 10 after
583 kSourceSsrc, kLastSr, kDelayLastSr)).Times(1); 582 kSourceSsrc, kLastSr, kDelayLastSr)).Times(1);
584 583
585 RtcpParser rtcp_parser(p.Data(), p.Length()); 584 RtcpParser rtcp_parser(p.Data(), p.Length());
586 rtcp_receiver.IncomingRtcpPacket(&rtcp_parser); 585 rtcp_receiver.IncomingRtcpPacket(&rtcp_parser);
587 586
588 EXPECT_TRUE(cast_log_verification.OnReceivedReceiverLogCalled()); 587 EXPECT_TRUE(cast_log_verification.OnReceivedReceiverLogCalled());
589 } 588 }
590 589
591 } // namespace cast 590 } // namespace cast
592 } // namespace media 591 } // namespace media
OLDNEW
« no previous file with comments | « media/cast/rtcp/receiver_rtcp_event_subscriber_unittest.cc ('k') | media/cast/rtcp/rtcp_sender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698