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

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

Issue 2635623002: cc: Remove calls to owning_layer_->id() in RenderSurfaceImpl (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/render_surface_impl.h" 5 #include "cc/layers/render_surface_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 10
(...skipping 331 matching lines...) Expand 10 before | Expand all | Expand 10 after
342 void RenderSurfaceImpl::ResetPropertyChangedFlags() { 342 void RenderSurfaceImpl::ResetPropertyChangedFlags() {
343 surface_property_changed_ = false; 343 surface_property_changed_ = false;
344 ancestor_property_changed_ = false; 344 ancestor_property_changed_ = false;
345 } 345 }
346 346
347 void RenderSurfaceImpl::ClearLayerLists() { 347 void RenderSurfaceImpl::ClearLayerLists() {
348 layer_list_.clear(); 348 layer_list_.clear();
349 } 349 }
350 350
351 int RenderSurfaceImpl::GetRenderPassId() { 351 int RenderSurfaceImpl::GetRenderPassId() {
352 return owning_layer_->id(); 352 return id();
353 } 353 }
354 354
355 void RenderSurfaceImpl::AppendRenderPasses(RenderPassSink* pass_sink) { 355 void RenderSurfaceImpl::AppendRenderPasses(RenderPassSink* pass_sink) {
356 std::unique_ptr<RenderPass> pass = RenderPass::Create(layer_list_.size()); 356 std::unique_ptr<RenderPass> pass = RenderPass::Create(layer_list_.size());
357 pass->SetNew(owning_layer_->id(), content_rect(), 357 pass->SetNew(id(), content_rect(),
358 gfx::IntersectRects(content_rect(), 358 gfx::IntersectRects(content_rect(),
359 damage_tracker_->current_damage_rect()), 359 damage_tracker_->current_damage_rect()),
360 draw_properties_.screen_space_transform); 360 draw_properties_.screen_space_transform);
361 pass->filters = Filters(); 361 pass->filters = Filters();
362 pass->background_filters = BackgroundFilters(); 362 pass->background_filters = BackgroundFilters();
363 pass_sink->AppendRenderPass(std::move(pass)); 363 pass_sink->AppendRenderPass(std::move(pass));
364 } 364 }
365 365
366 void RenderSurfaceImpl::AppendQuads(RenderPass* render_pass, 366 void RenderSurfaceImpl::AppendQuads(RenderPass* render_pass,
367 AppendQuadsData* append_quads_data) { 367 AppendQuadsData* append_quads_data) {
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
413 413
414 RenderPassDrawQuad* quad = 414 RenderPassDrawQuad* quad =
415 render_pass->CreateAndAppendDrawQuad<RenderPassDrawQuad>(); 415 render_pass->CreateAndAppendDrawQuad<RenderPassDrawQuad>();
416 quad->SetNew(shared_quad_state, content_rect(), visible_layer_rect, 416 quad->SetNew(shared_quad_state, content_rect(), visible_layer_rect,
417 GetRenderPassId(), mask_resource_id, mask_uv_scale, 417 GetRenderPassId(), mask_resource_id, mask_uv_scale,
418 mask_texture_size, owning_layer_to_target_scale, 418 mask_texture_size, owning_layer_to_target_scale,
419 FiltersOrigin()); 419 FiltersOrigin());
420 } 420 }
421 421
422 } // namespace cc 422 } // namespace cc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698