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

Unified Diff: media/base/video_frame_unittest.cc

Issue 175223003: HW Video: Make media::VideoFrame handle the sync point of the compositor as well as webgl (Closed) Base URL: https://git.chromium.org/chromium/src.git@master
Patch Set: Add WaitSyncPoint in WMPImpl::paint(...) 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/base/video_frame_unittest.cc
diff --git a/media/base/video_frame_unittest.cc b/media/base/video_frame_unittest.cc
index 05465d9b09dc1ec28751511bf97d205a71635378..243056b3558d5d2fdacee57b6abcee07ffe722f7 100644
--- a/media/base/video_frame_unittest.cc
+++ b/media/base/video_frame_unittest.cc
@@ -265,8 +265,14 @@ TEST(VideoFrame, TextureNoLongerNeededCallbackAfterTakingAndReleasingMailbox) {
EXPECT_EQ(sync_point, mailbox_holder->sync_point);
// Finish using the mailbox_holder and drop our reference.
+ uint32 previous_sync_point = 0;
+ sync_point = 1;
+ EXPECT_EQ(previous_sync_point, frame->SwapReleaseSyncPoint(sync_point));
+ // The last sync point will be used.
+ previous_sync_point = sync_point;
sync_point = 10;
- mailbox_holder->sync_point = sync_point;
+ EXPECT_EQ(previous_sync_point, frame->SwapReleaseSyncPoint(sync_point));
+
}
EXPECT_EQ(sync_point, called_sync_point);
}

Powered by Google App Engine
This is Rietveld 408576698