Refactor scroll updates during flexbox layout.

third_party/WebKit/Source/core/layout/LayoutBlock.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 325 matching lines @@
     // This function disables the 'overflow' check in inlineBlockBaseline.
     // For 'inline-block', CSS says that the baseline is the bottom margin edge
     // if 'overflow' is not visible. But some descendant classes want to ignore
     // this condition.
     virtual bool shouldIgnoreOverflowPropertyForInlineBlockBaseline() const { return false; }
 
     bool hitTestOverflowControl(HitTestResult&, const HitTestLocation&, const LayoutPoint& adjustedLocation) override;
     bool hitTestChildren(HitTestResult&, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, HitTestAction) override;
     void updateHitTestResult(HitTestResult&, const LayoutPoint&) override;
 
-    // Delay update scrollbar until finishDelayUpdateScrollInfo() will be
-    // called. This function is used when a flexbox is laying out its
-    // descendant. If multiple calls are made to startDelayUpdateScrollInfo(),
-    // finishDelayUpdateScrollInfo() will do nothing until finishDelayUpdateScrollInfo()
-    // is called the same number of times.
-    // finishDelayUpdateScrollInfo returns true when it marked something for layout.
-    // It will also return a map of saved scroll positions that the caller should restore
-    // on the given scrollable areas after performing the layout.
-    // This can be necessary because Flexbox's multi-pass layout can lose the scroll position.
-    // TODO(cbiesinger): This is a temporary hack. The right solution is to delay the scroll
-    // clamping that currently happens in PaintLayerScrollableArea::updateAfterLayout to only
-    // happen after all layout is done, i.e. during updateLayerPositionsAfterLayout. However,
-    // that currently fails a layout test. To fix this bug in time for M50, we use this temporary
-    // hack. The real fix is tracked in crbug.com/600036
-    typedef PersistentHeapHashMap<Member<PaintLayerScrollableArea>, DoublePoint> ScrollPositionMap;
-    static void startDelayUpdateScrollInfo();
-    static bool finishDelayUpdateScrollInfo(SubtreeLayoutScope*, ScrollPositionMap*);
-
     void updateAfterLayout();
 
     void styleWillChange(StyleDifference, const ComputedStyle& newStyle) override;
     void styleDidChange(StyleDifference, const ComputedStyle* oldStyle) override;
     void updateFromStyle() override;
 
     // Returns true if non-visible overflow should be respected. Otherwise hasOverflowClip() will be
     // false and we won't create scrollable area for this object even if overflow is non-visible.
     virtual bool allowsOverflowClip() const;
 
@@ skipping 123 matching lines @@
     // FIXME: This is temporary as we move code that accesses block flow
     // member variables out of LayoutBlock and into LayoutBlockFlow.
     friend class LayoutBlockFlow;
 };
 
 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutBlock, isLayoutBlock());
 
 } // namespace blink
 
 #endif // LayoutBlock_h