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

Side by Side Diff: media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer_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 "media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h" 5 #include "media/cast/transport/rtp_sender/rtp_packetizer/rtp_packetizer.h"
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "base/test/simple_test_tick_clock.h" 8 #include "base/test/simple_test_tick_clock.h"
9 #include "media/cast/logging/simple_event_subscriber.h" 9 #include "media/cast/logging/simple_event_subscriber.h"
10 #include "media/cast/test/fake_single_thread_task_runner.h" 10 #include "media/cast/test/fake_single_thread_task_runner.h"
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 uint32 expectd_rtp_timestamp_; 93 uint32 expectd_rtp_timestamp_;
94 94
95 DISALLOW_COPY_AND_ASSIGN(TestRtpPacketTransport); 95 DISALLOW_COPY_AND_ASSIGN(TestRtpPacketTransport);
96 }; 96 };
97 97
98 class RtpPacketizerTest : public ::testing::Test { 98 class RtpPacketizerTest : public ::testing::Test {
99 protected: 99 protected:
100 RtpPacketizerTest() 100 RtpPacketizerTest()
101 : task_runner_(new test::FakeSingleThreadTaskRunner(&testing_clock_)), 101 : task_runner_(new test::FakeSingleThreadTaskRunner(&testing_clock_)),
102 video_frame_(), 102 video_frame_(),
103 packet_storage_(&testing_clock_, kMaxPacketStorageTimeMs), 103 packet_storage_(&testing_clock_, kMaxPacketStorageTimeMs) {
104 logging_(GetLoggingConfigWithRawEventsAndStatsEnabled()) {
105 logging_.AddRawEventSubscriber(&subscriber_); 104 logging_.AddRawEventSubscriber(&subscriber_);
106 config_.sequence_number = kSeqNum; 105 config_.sequence_number = kSeqNum;
107 config_.ssrc = kSsrc; 106 config_.ssrc = kSsrc;
108 config_.payload_type = kPayload; 107 config_.payload_type = kPayload;
109 config_.max_payload_length = kMaxPacketLength; 108 config_.max_payload_length = kMaxPacketLength;
110 transport_.reset(new TestRtpPacketTransport(config_)); 109 transport_.reset(new TestRtpPacketTransport(config_));
111 pacer_.reset(new PacedSender( 110 pacer_.reset(new PacedSender(
112 &testing_clock_, &logging_, transport_.get(), task_runner_)); 111 &testing_clock_, &logging_, transport_.get(), task_runner_));
113 pacer_->RegisterVideoSsrc(config_.ssrc); 112 pacer_->RegisterVideoSsrc(config_.ssrc);
114 rtp_packetizer_.reset(new RtpPacketizer( 113 rtp_packetizer_.reset(new RtpPacketizer(
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 testing_clock_.NowTicks()); 183 testing_clock_.NowTicks());
185 RunTasks(33 + 1); 184 RunTasks(33 + 1);
186 EXPECT_EQ(expected_num_of_packets, rtp_packetizer_->send_packets_count()); 185 EXPECT_EQ(expected_num_of_packets, rtp_packetizer_->send_packets_count());
187 EXPECT_EQ(kFrameSize, rtp_packetizer_->send_octet_count()); 186 EXPECT_EQ(kFrameSize, rtp_packetizer_->send_octet_count());
188 EXPECT_EQ(expected_num_of_packets, transport_->number_of_packets_received()); 187 EXPECT_EQ(expected_num_of_packets, transport_->number_of_packets_received());
189 } 188 }
190 189
191 } // namespace transport 190 } // namespace transport
192 } // namespace cast 191 } // namespace cast
193 } // namespace media 192 } // namespace media
OLDNEW
« no previous file with comments | « media/cast/transport/pacing/paced_sender_unittest.cc ('k') | media/cast/video_receiver/video_decoder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698