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

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

Issue 2173563002: Improve GL shader YUV adjustment. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use enum hack Created 4 years, 4 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 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 115
116 if (external_resources.type == 116 if (external_resources.type ==
117 VideoFrameExternalResources::SOFTWARE_RESOURCE) { 117 VideoFrameExternalResources::SOFTWARE_RESOURCE) {
118 software_resources_ = external_resources.software_resources; 118 software_resources_ = external_resources.software_resources;
119 software_release_callback_ = 119 software_release_callback_ =
120 external_resources.software_release_callback; 120 external_resources.software_release_callback;
121 return true; 121 return true;
122 } 122 }
123 frame_resource_offset_ = external_resources.offset; 123 frame_resource_offset_ = external_resources.offset;
124 frame_resource_multiplier_ = external_resources.multiplier; 124 frame_resource_multiplier_ = external_resources.multiplier;
125 frame_bits_per_channel_ = external_resources.bits_per_channel;
125 126
126 DCHECK_EQ(external_resources.mailboxes.size(), 127 DCHECK_EQ(external_resources.mailboxes.size(),
127 external_resources.release_callbacks.size()); 128 external_resources.release_callbacks.size());
128 ResourceProvider::ResourceIdArray resource_ids; 129 ResourceProvider::ResourceIdArray resource_ids;
129 resource_ids.reserve(external_resources.mailboxes.size()); 130 resource_ids.reserve(external_resources.mailboxes.size());
130 for (size_t i = 0; i < external_resources.mailboxes.size(); ++i) { 131 for (size_t i = 0; i < external_resources.mailboxes.size(); ++i) {
131 unsigned resource_id = resource_provider->CreateResourceFromTextureMailbox( 132 unsigned resource_id = resource_provider->CreateResourceFromTextureMailbox(
132 external_resources.mailboxes[i], 133 external_resources.mailboxes[i],
133 SingleReleaseCallbackImpl::Create( 134 SingleReleaseCallbackImpl::Create(
134 external_resources.release_callbacks[i]), 135 external_resources.release_callbacks[i]),
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
276 277
277 YUVVideoDrawQuad* yuv_video_quad = 278 YUVVideoDrawQuad* yuv_video_quad =
278 render_pass->CreateAndAppendDrawQuad<YUVVideoDrawQuad>(); 279 render_pass->CreateAndAppendDrawQuad<YUVVideoDrawQuad>();
279 yuv_video_quad->SetNew( 280 yuv_video_quad->SetNew(
280 shared_quad_state, quad_rect, opaque_rect, visible_quad_rect, 281 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, 282 ya_tex_coord_rect, uv_tex_coord_rect, ya_tex_size, uv_tex_size,
282 frame_resources_[0].id, frame_resources_[1].id, 283 frame_resources_[0].id, frame_resources_[1].id,
283 frame_resources_.size() > 2 ? frame_resources_[2].id 284 frame_resources_.size() > 2 ? frame_resources_[2].id
284 : frame_resources_[1].id, 285 : frame_resources_[1].id,
285 frame_resources_.size() > 3 ? frame_resources_[3].id : 0, color_space, 286 frame_resources_.size() > 3 ? frame_resources_[3].id : 0, color_space,
286 frame_resource_offset_, frame_resource_multiplier_); 287 frame_resource_offset_, frame_resource_multiplier_,
288 frame_bits_per_channel_);
287 ValidateQuadResources(yuv_video_quad); 289 ValidateQuadResources(yuv_video_quad);
288 break; 290 break;
289 } 291 }
290 case VideoFrameExternalResources::RGBA_RESOURCE: 292 case VideoFrameExternalResources::RGBA_RESOURCE:
291 case VideoFrameExternalResources::RGBA_PREMULTIPLIED_RESOURCE: 293 case VideoFrameExternalResources::RGBA_PREMULTIPLIED_RESOURCE:
292 case VideoFrameExternalResources::RGB_RESOURCE: { 294 case VideoFrameExternalResources::RGB_RESOURCE: {
293 DCHECK_EQ(frame_resources_.size(), 1u); 295 DCHECK_EQ(frame_resources_.size(), 1u);
294 if (frame_resources_.size() < 1u) 296 if (frame_resources_.size() < 1u)
295 break; 297 break;
296 bool premultiplied_alpha = 298 bool premultiplied_alpha =
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
399 void VideoLayerImpl::SetNeedsRedraw() { 401 void VideoLayerImpl::SetNeedsRedraw() {
400 SetUpdateRect(gfx::UnionRects(update_rect(), gfx::Rect(bounds()))); 402 SetUpdateRect(gfx::UnionRects(update_rect(), gfx::Rect(bounds())));
401 layer_tree_impl()->SetNeedsRedraw(); 403 layer_tree_impl()->SetNeedsRedraw();
402 } 404 }
403 405
404 const char* VideoLayerImpl::LayerTypeAsString() const { 406 const char* VideoLayerImpl::LayerTypeAsString() const {
405 return "cc::VideoLayerImpl"; 407 return "cc::VideoLayerImpl";
406 } 408 }
407 409
408 } // namespace cc 410 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698