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

Side by Side Diff: cc/trees/property_tree_builder.cc

Issue 2032213002: cc: Put to_target and to_screen behind an accessor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rename ids to node_ids Created 4 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/trees/property_tree_builder.h" 5 #include "cc/trees/property_tree_builder.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 489 matching lines...) Expand 10 before | Expand all | Expand 10 after
500 if (layer == data_from_ancestor.page_scale_layer) 500 if (layer == data_from_ancestor.page_scale_layer)
501 data_for_children->in_subtree_of_page_scale_layer = true; 501 data_for_children->in_subtree_of_page_scale_layer = true;
502 node->data.in_subtree_of_page_scale_layer = 502 node->data.in_subtree_of_page_scale_layer =
503 data_for_children->in_subtree_of_page_scale_layer; 503 data_for_children->in_subtree_of_page_scale_layer;
504 504
505 // Surfaces inherently flatten transforms. 505 // Surfaces inherently flatten transforms.
506 data_for_children->should_flatten = 506 data_for_children->should_flatten =
507 ShouldFlattenTransform(layer) || has_surface; 507 ShouldFlattenTransform(layer) || has_surface;
508 DCHECK_GT(data_from_ancestor.property_trees->effect_tree.size(), 0u); 508 DCHECK_GT(data_from_ancestor.property_trees->effect_tree.size(), 0u);
509 509
510 node->data.target_id = data_for_children->property_trees->effect_tree 510 data_for_children->property_trees->transform_tree.SetTargetId(
511 .Node(data_from_ancestor.render_target) 511 node->id, data_for_children->property_trees->effect_tree
512 ->data.transform_id; 512 .Node(data_from_ancestor.render_target)
513 node->data.content_target_id = data_for_children->property_trees->effect_tree 513 ->data.transform_id);
514 .Node(data_for_children->render_target) 514 data_for_children->property_trees->transform_tree.SetContentTargetId(
515 ->data.transform_id; 515 node->id, data_for_children->property_trees->effect_tree
516 DCHECK_NE(node->data.target_id, kInvalidPropertyTreeNodeId); 516 .Node(data_for_children->render_target)
517 ->data.transform_id);
518 DCHECK_NE(
519 data_for_children->property_trees->transform_tree.TargetId(node->id),
520 kInvalidPropertyTreeNodeId);
517 521
518 node->data.has_potential_animation = has_potentially_animated_transform; 522 node->data.has_potential_animation = has_potentially_animated_transform;
519 node->data.is_currently_animating = layer->TransformIsAnimating(); 523 node->data.is_currently_animating = layer->TransformIsAnimating();
520 if (has_potentially_animated_transform) { 524 if (has_potentially_animated_transform) {
521 float maximum_animation_target_scale = 0.f; 525 float maximum_animation_target_scale = 0.f;
522 if (layer->MaximumTargetScale(&maximum_animation_target_scale)) { 526 if (layer->MaximumTargetScale(&maximum_animation_target_scale)) {
523 node->data.local_maximum_animation_target_scale = 527 node->data.local_maximum_animation_target_scale =
524 maximum_animation_target_scale; 528 maximum_animation_target_scale;
525 } 529 }
526 530
(...skipping 793 matching lines...) Expand 10 before | Expand all | Expand 10 after
1320 if (SkColorGetA(color) != 255) 1324 if (SkColorGetA(color) != 255)
1321 color = SkColorSetA(color, 255); 1325 color = SkColorSetA(color, 255);
1322 BuildPropertyTreesTopLevelInternal( 1326 BuildPropertyTreesTopLevelInternal(
1323 root_layer, page_scale_layer, inner_viewport_scroll_layer, 1327 root_layer, page_scale_layer, inner_viewport_scroll_layer,
1324 outer_viewport_scroll_layer, overscroll_elasticity_layer, 1328 outer_viewport_scroll_layer, overscroll_elasticity_layer,
1325 elastic_overscroll, page_scale_factor, device_scale_factor, viewport, 1329 elastic_overscroll, page_scale_factor, device_scale_factor, viewport,
1326 device_transform, property_trees, color); 1330 device_transform, property_trees, color);
1327 } 1331 }
1328 1332
1329 } // namespace cc 1333 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698