Exclude position:absolute elements when picking an anchor for ScrollAnchoring

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/layout/ScrollAnchor.h"
 
 #include "core/frame/FrameView.h"
 #include "core/frame/UseCounter.h"
 #include "core/layout/line/InlineTextBox.h"
 #include "core/paint/PaintLayerScrollableArea.h"
@@ skipping 86 matching lines @@
     if (scroller->isFrameView())
         relativeBounds.moveBy(-flooredIntPoint(scroller->scrollPositionDouble()));
     return relativeBounds;
 }
 
 static LayoutPoint computeRelativeOffset(const LayoutObject* layoutObject, const ScrollableArea* scroller, Corner corner)
 {
     return cornerPointOfRect(relativeBounds(layoutObject, scroller), corner);
 }
 
-static bool candidateMovesWithScroller(const LayoutObject* candidate, const ScrollableArea* scroller)
+static bool candidateMayMoveWithScroller(const LayoutObject* candidate, const ScrollableArea* scroller)
 {
-    if (candidate->style() && candidate->style()->hasViewportConstrainedPosition())
-        return false;
+    if (const ComputedStyle* style = candidate->style()) {
+        if (style->hasViewportConstrainedPosition())
+            return false;
+
+        if (style->hasOutOfFlowPosition()) {
+            // Absolute positioned elements with non-zero scrollTop/Left/Bottom/
+            // Right can stick to the viewport.
+            if (!style->top().isZero())

[skobes, 2016/06/01 04:38:55]

nit: this would be more concise with ||

+                return false;
+            if (!style->left().isZero())
+                return false;
+            if (!style->bottom().isZero())
+                return false;
+            if (!style->right().isZero())
+                return false;
+        }
+    }
 
     bool skippedByContainerLookup = false;
     candidate->container(scrollerLayoutBox(scroller), &skippedByContainerLookup);
     return !skippedByContainerLookup;
 }
 
 ScrollAnchor::ExamineResult ScrollAnchor::examine(const LayoutObject* candidate) const
 {
     if (candidate->isLayoutInline())
         return ExamineResult(Continue);
 
     if (!candidate->isText() && !candidate->isBox())
         return ExamineResult(Skip);
 
-    if (!candidateMovesWithScroller(candidate, m_scroller))
+    if (!candidateMayMoveWithScroller(candidate, m_scroller))
         return ExamineResult(Skip);
 
     LayoutRect candidateRect = relativeBounds(candidate, m_scroller);
     LayoutRect visibleRect = scrollerLayoutBoxItem(m_scroller).overflowClipRect(LayoutPoint());
 
     bool occupiesSpace = candidateRect.width() > 0 && candidateRect.height() > 0;
     if (occupiesSpace && visibleRect.intersects(candidateRect)) {
         return ExamineResult(
             visibleRect.contains(candidateRect) ? Return : Constrain,
             cornerFromCandidateRect(candidate));
@@ skipping 45 matching lines @@
         return;
 
     m_current.m_anchorObject->setIsScrollAnchorObject();
     m_current.m_savedRelativeOffset = computeRelativeOffset(
         m_current.m_anchorObject, m_scroller, m_current.m_corner);
 
     if (m_lastAdjusted) {
         // We need to update m_lastAdjusted.m_savedRelativeOffset, since it is
         // relative to the visible rect and the user may have scrolled since the
         // last adjustment.
-        if (!candidateMovesWithScroller(m_lastAdjusted.m_anchorObject, m_scroller)) {
+        if (!candidateMayMoveWithScroller(m_lastAdjusted.m_anchorObject, m_scroller)) {
             m_lastAdjusted.clear();
         } else if (m_lastAdjusted.m_anchorObject == m_current.m_anchorObject
             && m_lastAdjusted.m_corner == m_current.m_corner) {
             m_lastAdjusted.m_savedRelativeOffset = m_current.m_savedRelativeOffset;
         } else {
             m_lastAdjusted.m_savedRelativeOffset = computeRelativeOffset(
                 m_lastAdjusted.m_anchorObject, m_scroller, m_lastAdjusted.m_corner);
         }
     }
 }
@@ skipping 92 matching lines @@
 
 void ScrollAnchor::notifyRemoved(LayoutObject* layoutObject)
 {
     if (m_current.m_anchorObject == layoutObject)
         m_current.clear();
     if (m_lastAdjusted.m_anchorObject == layoutObject)
         m_lastAdjusted.clear();
 }
 
 } // namespace blink