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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/trees/layer_tree_host_impl.h

Issue 2824693002: Refactor LayerTreeImpl's scrollbar map to be keyed on element ids (Closed)
Patch Set: Address reviewer comments Created 3 years, 8 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
« cc/input/single_scrollbar_animation_controller_thinning_unittest.cc ('K') | « cc/layers/scrollbar_layer_unittest.cc ('k') | cc/trees/layer_tree_host_impl.cc » ('j') | cc/trees/layer_tree_impl.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_impl.h
diff --git a/cc/trees/layer_tree_host_impl.h b/cc/trees/layer_tree_host_impl.h
index b3893b08944c7ddb3daac6bb59adda919d7b731f..fd28bcd18c7c83b4b935e5a14be9e01b63e5fd18 100644
--- a/cc/trees/layer_tree_host_impl.h
+++ b/cc/trees/layer_tree_host_impl.h
@@ -321,8 +321,7 @@ class CC_EXPORT LayerTreeHostImpl
size_t SourceAnimationFrameNumberForTesting() const;
- void RegisterScrollbarAnimationController(int scroll_layer_id,
- ElementId scroll_element_id);
+ void RegisterScrollbarAnimationController(ElementId scroll_element_id);
void UnregisterScrollbarAnimationController(ElementId scroll_element_id);
ScrollbarAnimationController* ScrollbarAnimationControllerForElementId(
ElementId scroll_element_id) const;
@@ -355,7 +354,7 @@ class CC_EXPORT LayerTreeHostImpl
base::TimeDelta delay) override;
void SetNeedsAnimateForScrollbarAnimation() override;
void SetNeedsRedrawForScrollbarAnimation() override;
- ScrollbarSet ScrollbarsFor(int scroll_layer_id) const override;
+ ScrollbarSet ScrollbarsFor(ElementId scroll_element_id) const override;
void DidChangeScrollbarVisibility() override;
// VideoBeginFrameSource implementation.
« cc/input/single_scrollbar_animation_controller_thinning_unittest.cc ('K') | « cc/layers/scrollbar_layer_unittest.cc ('k') | cc/trees/layer_tree_host_impl.cc » ('j') | cc/trees/layer_tree_impl.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698