[css-flexbox] Add a missing needsLayout() check

Add a missing needsLayout() check

Without this, we will sometimes update our cache without having forced a new layout,
which means it will contain a length that still has the override size applied, which
is bad.

BUG=582212, 576859, 582151
R=leviw@chromium.org

Committed: https://crrev.com/8b1f305aa32f871dd7dbda0da422adc63598ca3b
Cr-Commit-Position: refs/heads/master@{#372425}

[cbiesinger, 2016-01-29 19:46:46]

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

[commit-bot: I haz the power, 2016-01-29 19:49:05]

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

[cbiesinger, 2016-01-29 19:49:14]

cbiesinger@chromium.org changed reviewers:
+ esprehn@chromium.org, skobes@chromium.org, szager@google.com

[cbiesinger, 2016-01-29 19:49:14]

Adding some reviewers in a timezone where it's not Saturday...

[skobes, 2016-01-29 19:58:20]

lgtm

[cbiesinger, 2016-01-29 20:10:14]

The CQ bit was unchecked by cbiesinger@chromium.org

[cbiesinger, 2016-01-29 20:10:24]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2016-01-29 20:11:04]

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

[cbiesinger, 2016-01-29 20:39:01]

Description was changed from

==========
Add a missing needsLayout() check

Without this, we will sometimes update our cache without having forced a new
layout,
which means it will contain a length that still has the override size applied,
which
is bad.

BUG=582212,576859,582151

R=leviw@chromium.org
==========

to

==========
Add a missing needsLayout() check

Without this, we will sometimes update our cache without having forced a new
layout,
which means it will contain a length that still has the override size applied,
which
is bad.

BUG=582212,576859,582151

R=leviw@chromium.org
==========

[commit-bot: I haz the power, 2016-01-29 21:26:08]

Description was changed from

==========
Add a missing needsLayout() check

Without this, we will sometimes update our cache without having forced a new
layout,
which means it will contain a length that still has the override size applied,
which
is bad.

BUG=582212,576859,582151

R=leviw@chromium.org
==========

to

==========
Add a missing needsLayout() check

Without this, we will sometimes update our cache without having forced a new
layout,
which means it will contain a length that still has the override size applied,
which
is bad.

BUG=582212,576859,582151

R=leviw@chromium.org
==========

[commit-bot: I haz the power, 2016-01-29 21:26:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-29 21:27:16]

Description was changed from

==========
Add a missing needsLayout() check

Without this, we will sometimes update our cache without having forced a new
layout,
which means it will contain a length that still has the override size applied,
which
is bad.

BUG=582212,576859,582151

R=leviw@chromium.org
==========

to

==========
Add a missing needsLayout() check

Without this, we will sometimes update our cache without having forced a new
layout,
which means it will contain a length that still has the override size applied,
which
is bad.

BUG=582212,576859,582151

R=leviw@chromium.org

Committed: https://crrev.com/8b1f305aa32f871dd7dbda0da422adc63598ca3b
Cr-Commit-Position: refs/heads/master@{#372425}
==========

[commit-bot: I haz the power, 2016-01-29 21:27:17]

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

[leviw_travelin_and_unemployed, 2016-01-29 21:42:13]

w00t.

LGTM.

Thanks, skobes :)