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

Unified Diff: cc/quads/yuv_video_draw_quad.cc

Issue 12157002: Adding YUVA support for enabling Alpha Playback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merging DrawYUVAVideoQuad with DrawYUVVideoQuad Created 7 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
Index: cc/quads/yuv_video_draw_quad.cc
diff --git a/cc/quads/yuv_video_draw_quad.cc b/cc/quads/yuv_video_draw_quad.cc
index 7876861bd878578bf38d9e91a4d3db3fb4c9f024..841959952159871b0e0a48fdfcf1fe43ba03e5b9 100644
--- a/cc/quads/yuv_video_draw_quad.cc
+++ b/cc/quads/yuv_video_draw_quad.cc
@@ -24,6 +24,33 @@ void YUVVideoDrawQuad::SetNew(const SharedQuadState* shared_quad_state,
const VideoLayerImpl::FramePlane& v_plane) {
gfx::Rect visible_rect = rect;
bool needs_blending = false;
+ SetAll(shared_quad_state, rect, opaque_rect, visible_rect, needs_blending,
+ tex_scale, y_plane, u_plane, v_plane, a_plane);
+}
+
+void YUVVideoDrawQuad::SetNew(const SharedQuadState* shared_quad_state,
+ gfx::Rect rect,
+ gfx::Rect opaque_rect,
+ gfx::SizeF tex_scale,
+ const VideoLayerImpl::FramePlane& y_plane,
+ const VideoLayerImpl::FramePlane& u_plane,
+ const VideoLayerImpl::FramePlane& v_plane,
+ const VideoLayerImpl::FramePlane& a_plane) {
+ gfx::Rect visible_rect = rect;
+ bool needs_blending = true;
+ SetAll(shared_quad_state, rect, opaque_rect, visible_rect, needs_blending,
+ tex_scale, y_plane, u_plane, v_plane, a_plane);
+}
+
+void YUVVideoDrawQuad::SetAll(const SharedQuadState* shared_quad_state,
+ gfx::Rect rect,
+ gfx::Rect opaque_rect,
+ gfx::Rect visible_rect,
+ bool needs_blending,
+ gfx::SizeF tex_scale,
+ const VideoLayerImpl::FramePlane& y_plane,
+ const VideoLayerImpl::FramePlane& u_plane,
+ const VideoLayerImpl::FramePlane& v_plane) {
DrawQuad::SetAll(shared_quad_state, DrawQuad::YUV_VIDEO_CONTENT, rect,
opaque_rect, visible_rect, needs_blending);
this->tex_scale = tex_scale;
@@ -40,13 +67,15 @@ void YUVVideoDrawQuad::SetAll(const SharedQuadState* shared_quad_state,
gfx::SizeF tex_scale,
const VideoLayerImpl::FramePlane& y_plane,
const VideoLayerImpl::FramePlane& u_plane,
- const VideoLayerImpl::FramePlane& v_plane) {
+ const VideoLayerImpl::FramePlane& v_plane,
+ const VideoLayerImpl::FramePlane& a_plane) {
DrawQuad::SetAll(shared_quad_state, DrawQuad::YUV_VIDEO_CONTENT, rect,
opaque_rect, visible_rect, needs_blending);
this->tex_scale = tex_scale;
this->y_plane = y_plane;
this->u_plane = u_plane;
this->v_plane = v_plane;
+ this->a_plane = a_plane;
}
void YUVVideoDrawQuad::IterateResources(
@@ -54,6 +83,9 @@ void YUVVideoDrawQuad::IterateResources(
y_plane.resource_id = callback.Run(y_plane.resource_id);
u_plane.resource_id = callback.Run(u_plane.resource_id);
v_plane.resource_id = callback.Run(v_plane.resource_id);
+ if(a_plane.resource_id) {
danakj 2013/04/04 15:01:55 nit: no {} for single-line body
vignesh 2013/04/04 22:06:14 Done.
+ a_plane.resource_id = callback.Run(a_plane.resource_id);
+ }
}
const YUVVideoDrawQuad* YUVVideoDrawQuad::MaterialCast(

Powered by Google App Engine
This is Rietveld 408576698