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

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

Issue 178133005: Audit/fix use of media::VideoFrame::coded_size() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 5c04e7a7 DCHECK fix. 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..e7efc38cbed99b24ff58c68bc43cc39d008c1722 100644
--- a/media/cast/test/end2end_unittest.cc
+++ b/media/cast/test/end2end_unittest.cc
@@ -362,8 +362,9 @@ class TestReceiverVideoCallback
<< "time_since_capture - upper_bound == "
<< (time_since_capture - upper_bound).InMilliseconds() << " mS";
EXPECT_LE(expected_video_frame.capture_time, render_time);
- EXPECT_EQ(expected_video_frame.width, video_frame->coded_size().width());
- EXPECT_EQ(expected_video_frame.height, video_frame->coded_size().height());
+ EXPECT_EQ(expected_video_frame.width, video_frame->visible_rect().width());
+ EXPECT_EQ(expected_video_frame.height,
+ video_frame->visible_rect().height());
gfx::Size size(expected_video_frame.width, expected_video_frame.height);
scoped_refptr<media::VideoFrame> expected_I420_frame =

Powered by Google App Engine
This is Rietveld 408576698