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

Unified Diff: cc/trees/layer_tree_impl.cc

Issue 2087963003: cc: Stop creating unused 0 property tree nodes other than transform Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: cc/trees/layer_tree_impl.cc
diff --git a/cc/trees/layer_tree_impl.cc b/cc/trees/layer_tree_impl.cc
index 797eaf91676150ce28177d8cb7d3fb766ca8ca91..955b395f2a13b4355d1b80c928515440ee457a9c 100644
--- a/cc/trees/layer_tree_impl.cc
+++ b/cc/trees/layer_tree_impl.cc
@@ -145,7 +145,7 @@ void LayerTreeImpl::DidUpdateScrollOffset(int layer_id, int transform_id) {
return;
}
- if (transform_id != -1) {
+ if (transform_id != TransformTree::kInvalidNodeId) {
TransformNode* node = transform_tree.Node(transform_id);
if (node->data.scroll_offset !=
scroll_tree.current_scroll_offset(layer_id)) {
@@ -557,7 +557,8 @@ void LayerTreeImpl::SetCurrentlyScrollingLayer(LayerImpl* layer) {
ScrollNode* scroll_node = scroll_tree.CurrentlyScrollingNode();
int old_id = scroll_node ? scroll_node->owner_id : Layer::INVALID_ID;
int new_id = layer ? layer->id() : Layer::INVALID_ID;
- int new_scroll_node_id = layer ? layer->scroll_tree_index() : -1;
+ int new_scroll_node_id =
+ layer ? layer->scroll_tree_index() : ScrollTree::kInvalidNodeId;
if (layer)
last_scrolled_layer_id_ = new_id;
@@ -1624,7 +1625,7 @@ static bool PointIsClippedByAncestorClipNode(
// ancestor render surface.
// We first check if the point is clipped by viewport.
- const ClipNode* clip_node = clip_tree.Node(1);
+ const ClipNode* clip_node = clip_tree.Node(ClipTree::kViewportNodeId);
gfx::Rect combined_clip_in_target_space =
gfx::ToEnclosingRect(clip_node->data.combined_clip_in_target_space);
if (!PointHitsRect(screen_space_point, gfx::Transform(),
@@ -1632,7 +1633,8 @@ static bool PointIsClippedByAncestorClipNode(
return true;
for (const ClipNode* clip_node = clip_tree.Node(layer->clip_tree_index());
- clip_node->id > 1; clip_node = clip_tree.parent(clip_node)) {
+ clip_node->id > ClipTree::kViewportNodeId;
+ clip_node = clip_tree.parent(clip_node)) {
if (clip_node->data.applies_local_clip) {
const TransformNode* transform_node =
transform_tree.Node(clip_node->data.target_id);
@@ -1641,10 +1643,11 @@ static bool PointIsClippedByAncestorClipNode(
const LayerImpl* target_layer =
layer->layer_tree_impl()->LayerById(transform_node->owner_id);
- DCHECK(transform_node->id == 0 || target_layer->render_surface() ||
+ DCHECK(transform_node->id == TransformTree::kDeviceNodeId ||
+ target_layer->render_surface() ||
layer->layer_tree_impl()->is_in_resourceless_software_draw_mode());
gfx::Transform surface_screen_space_transform =
- transform_node->id == 0 ||
+ transform_node->id == TransformTree::kDeviceNodeId ||
(layer->layer_tree_impl()
->is_in_resourceless_software_draw_mode())
? gfx::Transform()

Powered by Google App Engine
This is Rietveld 408576698