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

Side by Side Diff: cc/layers/video_layer_impl.cc

Issue 2919863004: Add flag to signal that video frame must be drawn using an overlay. (Closed)
Patch Set: cleanup Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/layers/video_layer_impl.h" 5 #include "cc/layers/video_layer_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
278 render_pass->CreateAndAppendDrawQuad<YUVVideoDrawQuad>(); 278 render_pass->CreateAndAppendDrawQuad<YUVVideoDrawQuad>();
279 yuv_video_quad->SetNew( 279 yuv_video_quad->SetNew(
280 shared_quad_state, quad_rect, opaque_rect, visible_quad_rect, 280 shared_quad_state, quad_rect, opaque_rect, visible_quad_rect,
281 ya_tex_coord_rect, uv_tex_coord_rect, ya_tex_size, uv_tex_size, 281 ya_tex_coord_rect, uv_tex_coord_rect, ya_tex_size, uv_tex_size,
282 frame_resources_[0].id, frame_resources_[1].id, 282 frame_resources_[0].id, frame_resources_[1].id,
283 frame_resources_.size() > 2 ? frame_resources_[2].id 283 frame_resources_.size() > 2 ? frame_resources_[2].id
284 : frame_resources_[1].id, 284 : frame_resources_[1].id,
285 frame_resources_.size() > 3 ? frame_resources_[3].id : 0, color_space, 285 frame_resources_.size() > 3 ? frame_resources_[3].id : 0, color_space,
286 frame_->ColorSpace(), frame_resource_offset_, 286 frame_->ColorSpace(), frame_resource_offset_,
287 frame_resource_multiplier_, frame_bits_per_channel_); 287 frame_resource_multiplier_, frame_bits_per_channel_);
288 yuv_video_quad->require_overlay = frame_->metadata()->IsTrue(
289 media::VideoFrameMetadata::REQUIRE_OVERLAY);
288 ValidateQuadResources(yuv_video_quad); 290 ValidateQuadResources(yuv_video_quad);
289 break; 291 break;
290 } 292 }
291 case VideoFrameExternalResources::RGBA_RESOURCE: 293 case VideoFrameExternalResources::RGBA_RESOURCE:
292 case VideoFrameExternalResources::RGBA_PREMULTIPLIED_RESOURCE: 294 case VideoFrameExternalResources::RGBA_PREMULTIPLIED_RESOURCE:
293 case VideoFrameExternalResources::RGB_RESOURCE: { 295 case VideoFrameExternalResources::RGB_RESOURCE: {
294 DCHECK_EQ(frame_resources_.size(), 1u); 296 DCHECK_EQ(frame_resources_.size(), 1u);
295 if (frame_resources_.size() < 1u) 297 if (frame_resources_.size() < 1u)
296 break; 298 break;
297 bool premultiplied_alpha = 299 bool premultiplied_alpha =
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
374 void VideoLayerImpl::SetNeedsRedraw() { 376 void VideoLayerImpl::SetNeedsRedraw() {
375 SetUpdateRect(gfx::UnionRects(update_rect(), gfx::Rect(bounds()))); 377 SetUpdateRect(gfx::UnionRects(update_rect(), gfx::Rect(bounds())));
376 layer_tree_impl()->SetNeedsRedraw(); 378 layer_tree_impl()->SetNeedsRedraw();
377 } 379 }
378 380
379 const char* VideoLayerImpl::LayerTypeAsString() const { 381 const char* VideoLayerImpl::LayerTypeAsString() const {
380 return "cc::VideoLayerImpl"; 382 return "cc::VideoLayerImpl";
381 } 383 }
382 384
383 } // namespace cc 385 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698