Revert of Fully (?) fix overflow: auto with delayed scroll updates

Revert of Fully (?) fix overflow: auto with delayed scroll updates (patchset #5 id:80001 of https://codereview.chromium.org/1734203002/ )

Reason for revert:
Crashes in blink::LayoutObject::container. See https://crbug.com/525316.

Original issue's description:
> Fully (?) fix overflow: auto with delayed scroll updates
>
> The scroll delaying code, used by Flexbox, never lays out the
> parent after the child relayout. This is a problem especially
> because the layout can add/remove a horizontal scrollbar,
> which changes the height of the element. A vertical scrollbar
> has this problem less often, though it sometimes does, especially
> in case of flex items. As far as I can tell, this never worked
> correctly since the scroll delaying code existed.
>
> This patch changes it so that when we eventually do the scroll info
> updates, we mark the ancestor chain for layout (up to the outer flexbox)
> and then call layoutFlexItems again, to do the correct layout.
>
> See also the discussion thread at
> https://groups.google.com/a/chromium.org/d/topic/layout-dev/e7Wzvxu7qiA/discussion
>
> BUG=579401, 584363
>
> Committed: https://crrev.com/775378f6ac8293e20a393c5203f5431a4a679339
> Cr-Commit-Position: refs/heads/master@{#377826}

TBR=leviw@chromium.org,skobes@chromium.org,szager@chromium.org,cbiesinger@google.com,cbiesinger@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=579401, 584363
Committed: https://crrev.com/f331574002bdde48e2c8ee0a657f202252548f8e
Cr-Commit-Position: refs/heads/master@{#378382}

[lfg, 2016-02-29 23:45:05]

Created Revert of Fully (?) fix overflow: auto with delayed scroll updates

[commit-bot: I haz the power, 2016-02-29 23:45:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1746353002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1746353002/1

[leviw_travelin_and_unemployed, 2016-02-29 23:51:26]

The CQ bit was unchecked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2016-02-29 23:51:42]

I don't think this revert is the right thing to do. Let's discuss it on the bug.

[leviw_travelin_and_unemployed, 2016-03-01 00:22:55]

The CQ bit was checked by leviw@chromium.org

[leviw_travelin_and_unemployed, 2016-03-01 00:22:55]

lgtm

I've changed my tune after verifying this is the issue. Hopefully cbiesinger can
get us patch that fixes both issues :)

[commit-bot: I haz the power, 2016-03-01 00:23:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1746353002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1746353002/1

[commit-bot: I haz the power, 2016-03-01 03:13:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-01 03:14:29]

Description was changed from

==========
Revert of Fully (?) fix overflow: auto with delayed scroll updates (patchset #5
id:80001 of https://codereview.chromium.org/1734203002/ )

Reason for revert:
Crashes in blink::LayoutObject::container. See https://crbug.com/525316.

Original issue's description:
> Fully (?) fix overflow: auto with delayed scroll updates
> 
> The scroll delaying code, used by Flexbox, never lays out the
> parent after the child relayout. This is a problem especially
> because the layout can add/remove a horizontal scrollbar,
> which changes the height of the element. A vertical scrollbar
> has this problem less often, though it sometimes does, especially
> in case of flex items. As far as I can tell, this never worked
> correctly since the scroll delaying code existed.
> 
> This patch changes it so that when we eventually do the scroll info
> updates, we mark the ancestor chain for layout (up to the outer flexbox)
> and then call layoutFlexItems again, to do the correct layout.
> 
> See also the discussion thread at
>
https://groups.google.com/a/chromium.org/d/topic/layout-dev/e7Wzvxu7qiA/discu...
> 
> BUG=579401,584363
> 
> Committed: https://crrev.com/775378f6ac8293e20a393c5203f5431a4a679339
> Cr-Commit-Position: refs/heads/master@{#377826}

TBR=leviw@chromium.org,skobes@chromium.org,szager@chromium.org,cbiesinger@goo...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=579401,584363
==========

to

==========
Revert of Fully (?) fix overflow: auto with delayed scroll updates (patchset #5
id:80001 of https://codereview.chromium.org/1734203002/ )

Reason for revert:
Crashes in blink::LayoutObject::container. See https://crbug.com/525316.

Original issue's description:
> Fully (?) fix overflow: auto with delayed scroll updates
>
> The scroll delaying code, used by Flexbox, never lays out the
> parent after the child relayout. This is a problem especially
> because the layout can add/remove a horizontal scrollbar,
> which changes the height of the element. A vertical scrollbar
> has this problem less often, though it sometimes does, especially
> in case of flex items. As far as I can tell, this never worked
> correctly since the scroll delaying code existed.
>
> This patch changes it so that when we eventually do the scroll info
> updates, we mark the ancestor chain for layout (up to the outer flexbox)
> and then call layoutFlexItems again, to do the correct layout.
>
> See also the discussion thread at
>
https://groups.google.com/a/chromium.org/d/topic/layout-dev/e7Wzvxu7qiA/discu...
>
> BUG=579401,584363
>
> Committed: https://crrev.com/775378f6ac8293e20a393c5203f5431a4a679339
> Cr-Commit-Position: refs/heads/master@{#377826}

TBR=leviw@chromium.org,skobes@chromium.org,szager@chromium.org,cbiesinger@goo...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=579401,584363

Committed: https://crrev.com/f331574002bdde48e2c8ee0a657f202252548f8e
Cr-Commit-Position: refs/heads/master@{#378382}
==========

[commit-bot: I haz the power, 2016-03-01 03:14:30]

Patchset 1 (id:??) landed as
https://crrev.com/f331574002bdde48e2c8ee0a657f202252548f8e
Cr-Commit-Position: refs/heads/master@{#378382}