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

Unified Diff: cc/trees/property_tree.cc

Issue 2911463002: Unify the calculation of main thread offset of sticky element (Closed)
Patch Set: Address comments 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 side-by-side diff with in-line comments
Download patch
Index: cc/trees/property_tree.cc
diff --git a/cc/trees/property_tree.cc b/cc/trees/property_tree.cc
index 50a7a7dbc615cf24aa75a9df07d646cd0b979138..c81ca78531eddfd9058c81144cea0c7752633940 100644
--- a/cc/trees/property_tree.cc
+++ b/cc/trees/property_tree.cc
@@ -378,7 +378,6 @@ gfx::Vector2dF StickyPositionOffset(TransformTree* tree, TransformNode* node) {
scroll_position,
gfx::SizeF(property_trees.scroll_tree.scroll_clip_layer_bounds(
scroll_node->id)));
- gfx::Vector2dF layer_offset(sticky_data->main_thread_offset);
gfx::Vector2dF ancestor_sticky_box_offset;
if (sticky_data->nearest_node_shifting_sticky_box !=
@@ -463,7 +462,7 @@ gfx::Vector2dF StickyPositionOffset(TransformTree* tree, TransformNode* node) {
ancestor_sticky_box_offset + ancestor_containing_block_offset +
sticky_offset - sticky_box_rect.OffsetFromOrigin();
- return sticky_offset - layer_offset - node->source_to_parent -
+ return sticky_offset - node->sticky_main_thread_offset -
sticky_box_rect.OffsetFromOrigin();
}

Powered by Google App Engine
This is Rietveld 408576698