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

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

Issue 2860293002: Change cc::ElementId to be a uint64_t (Closed)
Patch Set: none Created 3 years, 7 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 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 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/layer_impl.h" 5 #include "cc/layers/layer_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 609 matching lines...) Expand 10 before | Expand all | Expand 10 after
620 else 620 else
621 return 1.f; 621 return 1.f;
622 } 622 }
623 623
624 const gfx::Transform& LayerImpl::Transform() const { 624 const gfx::Transform& LayerImpl::Transform() const {
625 DCHECK_NE(GetTransformTree().FindNodeFromOwningLayerId(id()), nullptr); 625 DCHECK_NE(GetTransformTree().FindNodeFromOwningLayerId(id()), nullptr);
626 return GetTransformTree().FindNodeFromOwningLayerId(id())->local; 626 return GetTransformTree().FindNodeFromOwningLayerId(id())->local;
627 } 627 }
628 628
629 void LayerImpl::SetElementId(ElementId element_id) { 629 void LayerImpl::SetElementId(ElementId element_id) {
630 if (element_id == element_id_) 630 if (element_id.id == element_id_.id)
631 return; 631 return;
632 632
633 TRACE_EVENT1(TRACE_DISABLED_BY_DEFAULT("compositor-worker"), 633 TRACE_EVENT1(TRACE_DISABLED_BY_DEFAULT("compositor-worker"),
634 "LayerImpl::SetElementId", "element", 634 "LayerImpl::SetElementId", "element", element_id.id);
635 element_id.AsValue().release());
636 635
637 layer_tree_impl_->RemoveFromElementMap(this); 636 layer_tree_impl_->RemoveFromElementMap(this);
638 element_id_ = element_id; 637 element_id_ = element_id;
639 layer_tree_impl_->AddToElementMap(this); 638 layer_tree_impl_->AddToElementMap(this);
640 639
641 SetNeedsPushProperties(); 640 SetNeedsPushProperties();
642 } 641 }
643 642
644 void LayerImpl::SetMutableProperties(uint32_t properties) { 643 void LayerImpl::SetMutableProperties(uint32_t properties) {
645 if (mutable_properties_ == properties) 644 if (mutable_properties_ == properties)
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
792 791
793 state->SetDouble("opacity", Opacity()); 792 state->SetDouble("opacity", Opacity());
794 793
795 MathUtil::AddToTracedValue("position", position_, state); 794 MathUtil::AddToTracedValue("position", position_, state);
796 795
797 state->SetInteger("draws_content", DrawsContent()); 796 state->SetInteger("draws_content", DrawsContent());
798 state->SetInteger("gpu_memory_usage", 797 state->SetInteger("gpu_memory_usage",
799 base::saturated_cast<int>(GPUMemoryUsageInBytes())); 798 base::saturated_cast<int>(GPUMemoryUsageInBytes()));
800 799
801 if (element_id_) 800 if (element_id_)
802 element_id_.AddToTracedValue(state); 801 state->SetInteger("element_id", element_id_.id);
803 802
804 if (mutable_properties_ != MutableProperty::kNone) 803 if (mutable_properties_ != MutableProperty::kNone)
805 state->SetInteger("mutable_properties", mutable_properties_); 804 state->SetInteger("mutable_properties", mutable_properties_);
806 805
807 MathUtil::AddToTracedValue("scroll_offset", CurrentScrollOffset(), state); 806 MathUtil::AddToTracedValue("scroll_offset", CurrentScrollOffset(), state);
808 807
809 if (!ScreenSpaceTransform().IsIdentity()) 808 if (!ScreenSpaceTransform().IsIdentity())
810 MathUtil::AddToTracedValue("screen_space_transform", ScreenSpaceTransform(), 809 MathUtil::AddToTracedValue("screen_space_transform", ScreenSpaceTransform(),
811 state); 810 state);
812 811
(...skipping 160 matching lines...) Expand 10 before | Expand all | Expand 10 after
973 972
974 ScrollTree& LayerImpl::GetScrollTree() const { 973 ScrollTree& LayerImpl::GetScrollTree() const {
975 return GetPropertyTrees()->scroll_tree; 974 return GetPropertyTrees()->scroll_tree;
976 } 975 }
977 976
978 TransformTree& LayerImpl::GetTransformTree() const { 977 TransformTree& LayerImpl::GetTransformTree() const {
979 return GetPropertyTrees()->transform_tree; 978 return GetPropertyTrees()->transform_tree;
980 } 979 }
981 980
982 } // namespace cc 981 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698