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

Unified Diff: media/cast/net/rtp/frame_buffer_unittest.cc

Issue 1515433002: Replace uses of raw uint32's with a type-checked RtpTimeTicks data type. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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/net/rtp/frame_buffer_unittest.cc
diff --git a/media/cast/net/rtp/frame_buffer_unittest.cc b/media/cast/net/rtp/frame_buffer_unittest.cc
index f13771a41c228571297c7c761cab08a9d888bbea..df8116eeb88157a60f949d941433bb18e5be4dfb 100644
--- a/media/cast/net/rtp/frame_buffer_unittest.cc
+++ b/media/cast/net/rtp/frame_buffer_unittest.cc
@@ -25,7 +25,7 @@ class FrameBufferTest : public ::testing::Test {
};
TEST_F(FrameBufferTest, OnePacketInsertSanity) {
- rtp_header_.rtp_timestamp = 3000;
+ rtp_header_.rtp_timestamp = RtpTimeTicks().Expand(UINT32_C(3000));
rtp_header_.is_key_frame = true;
rtp_header_.frame_id = 5;
rtp_header_.reference_frame_id = 5;
@@ -35,7 +35,7 @@ TEST_F(FrameBufferTest, OnePacketInsertSanity) {
EXPECT_EQ(EncodedFrame::KEY, frame.dependency);
EXPECT_EQ(5u, frame.frame_id);
EXPECT_EQ(5u, frame.referenced_frame_id);
- EXPECT_EQ(3000u, frame.rtp_timestamp);
+ EXPECT_EQ(3000u, frame.rtp_timestamp.lower_32_bits());
}
TEST_F(FrameBufferTest, EmptyBuffer) {

Powered by Google App Engine
This is Rietveld 408576698