Disable scroll anchoring is an element within the scroll changes its in-flow state

third_party/WebKit/Source/core/layout/LayoutObject.cpp

Index: third_party/WebKit/Source/core/layout/LayoutObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutObject.cpp b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
index 37f163851d52e143dff84248748b9b67fc700f65..798188a72e93af7f77a03ebc4dc44125fc7cc95c 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
@@ -1712,6 +1712,24 @@ static inline bool areCursorsEqual(const ComputedStyle* a, const ComputedStyle*
     return a->cursor() == b->cursor() && (a->cursors() == b->cursors() || areNonIdenticalCursorListsEqual(a, b));
 }
 
+void LayoutObject::setScrollAnchorDisablingStyleChangedOnAncestor()
+{
+    // Walk up the parent chain and find the first scrolling block to disable
+    // scroll anchoring on.
+    LayoutObject* object = this;
+    Element* viewportDefiningElement = document().viewportDefiningElement();
+    do {
+        object = object->parent();
+        if (object && object->isLayoutBlock()) {
+            LayoutBlock* block = toLayoutBlock(object);
+            if (block->hasOverflowClip() || block->node() == viewportDefiningElement) {
+                block->setScrollAnchorDisablingStyleChanged(true);
+                return;
+            }
+        }
+    } while (object);

[skobes, 2016/09/07 19:26:27]

This can be a normal while loop.

[ymalik, 2016/09/07 19:55:37]

Done.

+}
+
 void LayoutObject::styleDidChange(StyleDifference diff, const ComputedStyle* oldStyle)
 {
     if (s_affectsParentBlock)
@@ -1723,6 +1741,11 @@ void LayoutObject::styleDidChange(StyleDifference diff, const ComputedStyle* old
     if (diff.needsFullLayout()) {
         LayoutCounter::layoutObjectStyleChanged(*this, oldStyle, *m_style);
 
+        // If the in-flow state of an element is changed, disable scroll
+        // anchoring on the containing scroller.
+        if (oldStyle->hasOutOfFlowPosition() != m_style->hasOutOfFlowPosition())
+            setScrollAnchorDisablingStyleChangedOnAncestor();
+
         // If the object already needs layout, then setNeedsLayout won't do
         // any work. But if the containing block has changed, then we may need
         // to mark the new containing blocks for layout. The change that can