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

Unified Diff: third_party/WebKit/Source/core/layout/compositing/CompositingRequirementsUpdater.cpp

Issue 2845613002: Fix the bug that sticky element may not be correctly invalidated due to non-promotion (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: third_party/WebKit/Source/core/layout/compositing/CompositingRequirementsUpdater.cpp
diff --git a/third_party/WebKit/Source/core/layout/compositing/CompositingRequirementsUpdater.cpp b/third_party/WebKit/Source/core/layout/compositing/CompositingRequirementsUpdater.cpp
index 54502ea395576daf2d6d81f50626119604972e21..fccc30d12f46d88386055754c055dd54b208774a 100644
--- a/third_party/WebKit/Source/core/layout/compositing/CompositingRequirementsUpdater.cpp
+++ b/third_party/WebKit/Source/core/layout/compositing/CompositingRequirementsUpdater.cpp
@@ -373,6 +373,15 @@ void CompositingRequirementsUpdater::UpdateRecursive(
RecursionData child_recursion_data = current_recursion_data;
child_recursion_data.subtree_is_compositing_ = false;
+ if (layer->SticksToScroller() &&
+ layer->AncestorOverflowLayer()->ScrollsOverflow() &&
+ current_recursion_data.compositing_ancestor_ &&
+ current_recursion_data.compositing_ancestor_->AncestorScrollingLayer() &&
chrishtr 2017/05/10 19:29:36 Please add the comments explaining that this is no
yigu 2017/05/10 20:01:42 Done.
+ !current_recursion_data.compositing_ancestor_->AncestorScrollingLayer()
+ ->IsRootLayer()) {
+ reasons_to_composite |= kCompositingReasonScrollDependentPosition;
+ }
+
bool will_be_composited_or_squashed =
can_be_composited && RequiresCompositingOrSquashing(reasons_to_composite);
if (will_be_composited_or_squashed) {

Powered by Google App Engine
This is Rietveld 408576698