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

Unified Diff: media/cast/test/end2end_unittest.cc

Issue 178073004: Cast: IPC from browser to renderer to send packet events from transport to cast library. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed hubbe's comments Created 6 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: media/cast/test/end2end_unittest.cc
diff --git a/media/cast/test/end2end_unittest.cc b/media/cast/test/end2end_unittest.cc
index 82fc568d2d2b959c96cedc276c9e8f0008cd87a9..52524cd8fa3b84ec74287f36c8e8e6e62c11ed0f 100644
--- a/media/cast/test/end2end_unittest.cc
+++ b/media/cast/test/end2end_unittest.cc
@@ -484,6 +484,7 @@ class End2EndTest : public ::testing::Test {
testing_clock_,
transport_config_,
base::Bind(&UpdateCastTransportStatus),
+ base::Bind(&End2EndTest::LogRawEvents, base::Unretained(this)),
task_runner_,
&sender_to_receiver_));
@@ -513,7 +514,6 @@ class End2EndTest : public ::testing::Test {
}
virtual void TearDown() OVERRIDE {
-
cast_sender_.reset();
cast_receiver_.reset();
task_runner_->RunTasks();
@@ -545,6 +545,22 @@ class End2EndTest : public ::testing::Test {
EXPECT_EQ(result, STATUS_INITIALIZED);
}
+ void LogRawEvents(const std::vector<PacketEvent>& packet_events) {
+ EXPECT_FALSE(packet_events.empty());
+ for (std::vector<media::cast::PacketEvent>::const_iterator it =
+ packet_events.begin();
+ it != packet_events.end();
+ ++it) {
+ cast_environment_->Logging()->InsertPacketEvent(it->timestamp,
+ it->type,
+ it->rtp_timestamp,
+ it->frame_id,
+ it->packet_id,
+ it->max_packet_id,
+ it->size);
+ }
+ }
+
AudioReceiverConfig audio_receiver_config_;
VideoReceiverConfig video_receiver_config_;
AudioSenderConfig audio_sender_config_;

Powered by Google App Engine
This is Rietveld 408576698