[css-flexbox] freezeInflexibleItems has to skip out-of-flow boxes

[css-flexbox] freezeInflexibleItems has to skip out-of-flow boxes

Out-of-flow items should not be frozen. This would have problematic
effects when they have flex-grow/flex-shrink set because freezing
them would subtract them from our flex-grow/shrink total without
having been added.

This also adds some DCHECKs.

R=eae@chromium.org,dgrogan@chromium.org
BUG=628072
Committed: https://crrev.com/fa759873d12635a75bdbdc3d6d5967a13378f5a3
Cr-Commit-Position: refs/heads/master@{#405608}

[cbiesinger, 2016-07-14 18:47:02]

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

[commit-bot: I haz the power, 2016-07-14 18:47:31]

Dry run: 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-07-14 21:27:02]

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

[commit-bot: I haz the power, 2016-07-14 21:27:03]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[eae, 2016-07-14 22:00:58]

LGTM

[eae, 2016-07-14 22:01:20]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-07-14 22:02:18]

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-07-14 22:35:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-14 22:35:28]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-14 22:36:46]

Description was changed from

==========
[css-flexbox] freezeInflexibleItems has to skip out-of-flow boxes

Out-of-flow items should not be frozen. This would have problematic
effects when they have flex-grow/flex-shrink set because freezing
them would subtract them from our flex-grow/shrink total without
having been added.

This also adds some DCHECKs.

R=eae@chromium.org,dgrogan@chromium.org
BUG=628072
==========

to

==========
[css-flexbox] freezeInflexibleItems has to skip out-of-flow boxes

Out-of-flow items should not be frozen. This would have problematic
effects when they have flex-grow/flex-shrink set because freezing
them would subtract them from our flex-grow/shrink total without
having been added.

This also adds some DCHECKs.

R=eae@chromium.org,dgrogan@chromium.org
BUG=628072

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

[commit-bot: I haz the power, 2016-07-14 22:36:47]

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