[css-flexbox] Have to use forceChildLayout to force layout

[css-flexbox] Have to use forceChildLayout to force layout

The flexbox code was assuming that calling layoutIfNeeded()
will do a full layout if needsLayout() is true.

However, that is not the case. It is possible for needsLayout()
to return true because posChildNeedsLayout is true, in which
case calling layout() will not do the full layout that the
flexbox code expects. Instead, call forceChildLayout to
ensure that layout happens.

R=eae@chromium.org,dgrogan@chromium.org
CC=szager@chromium.org
BUG=633212, 629491
Committed: https://crrev.com/e929360ccd93a1e98db3b19929a68d1ed4fa4d5d
Cr-Commit-Position: refs/heads/master@{#410135}

[cbiesinger, 2016-08-04 19:48:34]

The CQ bit was checked by cbiesinger@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-04 19:49:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[cbiesinger, 2016-08-04 19:50:21]

A note on the testcase: the original bug report required resizing the window,
but I actually found a testcase that fails statically. I was unable to make a
resizing testcase work (as in, unable to make it fail in current Chrome) so I'm
going with this one.

[commit-bot: I haz the power, 2016-08-04 21:04:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-04 21:04:07]

Dry run: This issue passed the CQ dry run.

[cbiesinger, 2016-08-05 15:54:02]

Description was changed from

==========
[css-flexbox] Have to use forceChildLayout to force layout

The flexbox code was assuming that calling layoutIfNeeded()
will do a full layout if needsLayout() is true.

However, that is not the case. It is possible for needsLayout()
to return true because posChildNeedsLayout is true, in which
case calling layout() will not do the full layout that the
flexbox code expects. Instead, call forceChildLayout to
ensure that layout happens.

R=eae@chromium.org,dgrogan@chromium.org
CC=szager@chromium.org
BUG=633212
==========

to

==========
[css-flexbox] Have to use forceChildLayout to force layout

The flexbox code was assuming that calling layoutIfNeeded()
will do a full layout if needsLayout() is true.

However, that is not the case. It is possible for needsLayout()
to return true because posChildNeedsLayout is true, in which
case calling layout() will not do the full layout that the
flexbox code expects. Instead, call forceChildLayout to
ensure that layout happens.

R=eae@chromium.org,dgrogan@chromium.org
CC=szager@chromium.org
BUG=633212,629491
==========

[eae, 2016-08-05 19:10:03]

LGTM

[cbiesinger, 2016-08-05 19:12:17]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2016-08-05 19:12:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-05 19:17:44]

Description was changed from

==========
[css-flexbox] Have to use forceChildLayout to force layout

The flexbox code was assuming that calling layoutIfNeeded()
will do a full layout if needsLayout() is true.

However, that is not the case. It is possible for needsLayout()
to return true because posChildNeedsLayout is true, in which
case calling layout() will not do the full layout that the
flexbox code expects. Instead, call forceChildLayout to
ensure that layout happens.

R=eae@chromium.org,dgrogan@chromium.org
CC=szager@chromium.org
BUG=633212,629491
==========

to

==========
[css-flexbox] Have to use forceChildLayout to force layout

The flexbox code was assuming that calling layoutIfNeeded()
will do a full layout if needsLayout() is true.

However, that is not the case. It is possible for needsLayout()
to return true because posChildNeedsLayout is true, in which
case calling layout() will not do the full layout that the
flexbox code expects. Instead, call forceChildLayout to
ensure that layout happens.

R=eae@chromium.org,dgrogan@chromium.org
CC=szager@chromium.org
BUG=633212,629491
==========

[commit-bot: I haz the power, 2016-08-05 19:17:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-05 19:21:06]

Description was changed from

==========
[css-flexbox] Have to use forceChildLayout to force layout

The flexbox code was assuming that calling layoutIfNeeded()
will do a full layout if needsLayout() is true.

However, that is not the case. It is possible for needsLayout()
to return true because posChildNeedsLayout is true, in which
case calling layout() will not do the full layout that the
flexbox code expects. Instead, call forceChildLayout to
ensure that layout happens.

R=eae@chromium.org,dgrogan@chromium.org
CC=szager@chromium.org
BUG=633212,629491
==========

to

==========
[css-flexbox] Have to use forceChildLayout to force layout

The flexbox code was assuming that calling layoutIfNeeded()
will do a full layout if needsLayout() is true.

However, that is not the case. It is possible for needsLayout()
to return true because posChildNeedsLayout is true, in which
case calling layout() will not do the full layout that the
flexbox code expects. Instead, call forceChildLayout to
ensure that layout happens.

R=eae@chromium.org,dgrogan@chromium.org
CC=szager@chromium.org
BUG=633212,629491

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

[commit-bot: I haz the power, 2016-08-05 19:21:08]

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