| Index: webrtc/media/engine/webrtcvideoframe_unittest.cc
|
| diff --git a/webrtc/media/engine/webrtcvideoframe_unittest.cc b/webrtc/media/engine/webrtcvideoframe_unittest.cc
|
| index 12c8bfe741e043801d3690856393dc2ea38f59a2..7849de61b0a8bf4914f7eb30aeddbb6e009f0416 100644
|
| --- a/webrtc/media/engine/webrtcvideoframe_unittest.cc
|
| +++ b/webrtc/media/engine/webrtcvideoframe_unittest.cc
|
| @@ -20,6 +20,7 @@ namespace {
|
|
|
| class WebRtcVideoTestFrame : public cricket::WebRtcVideoFrame {
|
| public:
|
| + // The ApplyRotationToFrame test needs this as a public method.
|
| using cricket::WebRtcVideoFrame::set_rotation;
|
|
|
| virtual VideoFrame* CreateEmptyFrame(int w,
|
| @@ -47,7 +48,7 @@ class WebRtcVideoFrameTest : public VideoFrameTest<cricket::WebRtcVideoFrame> {
|
| // Build the CapturedFrame.
|
| cricket::CapturedFrame captured_frame;
|
| captured_frame.fourcc = cricket::FOURCC_I420;
|
| - captured_frame.time_stamp = 5678;
|
| + captured_frame.time_stamp = rtc::TimeNanos();
|
| captured_frame.rotation = frame_rotation;
|
| captured_frame.width = frame_width;
|
| captured_frame.height = frame_height;
|
| @@ -66,7 +67,8 @@ class WebRtcVideoFrameTest : public VideoFrameTest<cricket::WebRtcVideoFrame> {
|
| apply_rotation));
|
|
|
| // Verify the new frame.
|
| - EXPECT_EQ(5678, frame.GetTimeStamp());
|
| + EXPECT_EQ(captured_frame.time_stamp / rtc::kNumNanosecsPerMicrosec,
|
| + frame.timestamp_us());
|
| if (apply_rotation)
|
| EXPECT_EQ(webrtc::kVideoRotation_0, frame.rotation());
|
| else
|
| @@ -271,13 +273,16 @@ TEST_F(WebRtcVideoFrameTest, TextureInitialValues) {
|
| webrtc::NativeHandleBuffer* buffer =
|
| new rtc::RefCountedObject<webrtc::test::FakeNativeHandleBuffer>(
|
| dummy_handle, 640, 480);
|
| - cricket::WebRtcVideoFrame frame(buffer, 200, webrtc::kVideoRotation_0);
|
| + // Timestamp is converted from ns to us, so last three digits are lost.
|
| + cricket::WebRtcVideoFrame frame(buffer, 20000, webrtc::kVideoRotation_0);
|
| EXPECT_EQ(dummy_handle, frame.GetNativeHandle());
|
| EXPECT_EQ(640, frame.width());
|
| EXPECT_EQ(480, frame.height());
|
| - EXPECT_EQ(200, frame.GetTimeStamp());
|
| - frame.SetTimeStamp(400);
|
| - EXPECT_EQ(400, frame.GetTimeStamp());
|
| + EXPECT_EQ(20000, frame.GetTimeStamp());
|
| + EXPECT_EQ(20, frame.timestamp_us());
|
| + frame.set_timestamp_us(40);
|
| + EXPECT_EQ(40000, frame.GetTimeStamp());
|
| + EXPECT_EQ(40, frame.timestamp_us());
|
| }
|
|
|
| TEST_F(WebRtcVideoFrameTest, CopyTextureFrame) {
|
| @@ -286,12 +291,14 @@ TEST_F(WebRtcVideoFrameTest, CopyTextureFrame) {
|
| webrtc::NativeHandleBuffer* buffer =
|
| new rtc::RefCountedObject<webrtc::test::FakeNativeHandleBuffer>(
|
| dummy_handle, 640, 480);
|
| - cricket::WebRtcVideoFrame frame1(buffer, 200, webrtc::kVideoRotation_0);
|
| + // Timestamp is converted from ns to us, so last three digits are lost.
|
| + cricket::WebRtcVideoFrame frame1(buffer, 20000, webrtc::kVideoRotation_0);
|
| cricket::VideoFrame* frame2 = frame1.Copy();
|
| EXPECT_EQ(frame1.GetNativeHandle(), frame2->GetNativeHandle());
|
| EXPECT_EQ(frame1.width(), frame2->width());
|
| EXPECT_EQ(frame1.height(), frame2->height());
|
| EXPECT_EQ(frame1.GetTimeStamp(), frame2->GetTimeStamp());
|
| + EXPECT_EQ(frame1.timestamp_us(), frame2->timestamp_us());
|
| delete frame2;
|
| }
|
|
|
|
|