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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/resources/video_resource_updater.cc

Issue 207233002: cc: Support texcoord offsets in YUVVideoDrawQuad (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 61f23173 Test update. Created 6 years, 9 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
« cc/output/renderer_pixeltest.cc ('K') | « cc/quads/yuv_video_draw_quad.cc ('k') | cc/test/data/yuv_stripes.png » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/video_resource_updater.cc
diff --git a/cc/resources/video_resource_updater.cc b/cc/resources/video_resource_updater.cc
index ec8b148c81c89447c898fd2e7a6a53d74544eb48..df428742c7ae8c72f9eb458120d0ed98501e8a34 100644
--- a/cc/resources/video_resource_updater.cc
+++ b/cc/resources/video_resource_updater.cc
@@ -58,12 +58,6 @@ VideoFrameExternalResources VideoResourceUpdater::
bool VideoResourceUpdater::VerifyFrame(
const scoped_refptr<media::VideoFrame>& video_frame) {
- // If these fail, we'll have to add logic that handles offset bitmap/texture
- // UVs. For now, just expect (0, 0) offset, since all our decoders so far
- // don't offset.
- DCHECK_EQ(video_frame->visible_rect().x(), 0);
- DCHECK_EQ(video_frame->visible_rect().y(), 0);
-
switch (video_frame->format()) {
// Acceptable inputs.
case media::VideoFrame::YV12:
« cc/output/renderer_pixeltest.cc ('K') | « cc/quads/yuv_video_draw_quad.cc ('k') | cc/test/data/yuv_stripes.png » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698