Make sure to save/restore scroll positions in finishDelayUpdateScrollInfo

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

Index: third_party/WebKit/Source/core/layout/LayoutBlock.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlock.h b/third_party/WebKit/Source/core/layout/LayoutBlock.h
index b24808b44da54b1f7ae0cdb9910cee75ad6351ee..c192d579fc4caf09d6d7b4b3c83044ee2fd6366c 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlock.h
+++ b/third_party/WebKit/Source/core/layout/LayoutBlock.h
@@ -339,8 +339,17 @@ protected:
     // 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 WTF::HashMap<PaintLayerScrollableArea*, DoublePoint> ScrollPositionMap;
     static void startDelayUpdateScrollInfo();
-    static bool finishDelayUpdateScrollInfo(SubtreeLayoutScope*);
+    static bool finishDelayUpdateScrollInfo(SubtreeLayoutScope*, ScrollPositionMap*);
 
     void updateScrollInfoAfterLayout();