[css-flexbox] Correctly implement freezing of inflexible items

[css-flexbox] Correctly implement freezing of inflexible items

We did not have a correct implementation of steps 2 and 3 of the
"9.7 Resolving flexible lengths" section of the spec:
https://drafts.csswg.org/css-flexbox/#resolve-flexible-lengths

In particular inflexible items (flex factor 0 or flex basis
violating the min/max size constraint) reduce the initial free
space.

This is an issue when sizing flex items with a flex factor of 0..1.

BUG=617956
TEST=css3/flexbox/max-width-violation.html

Committed: https://crrev.com/143408653a5c0328254ea06d0f3a825d6505a114
Cr-Commit-Position: refs/heads/master@{#402279}

[cbiesinger, 2016-06-24 22:07:28]

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

[commit-bot: I haz the power, 2016-06-24 22:07:56]

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-06-24 22:19:24]

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

[commit-bot: I haz the power, 2016-06-24 22:19:25]

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

[cbiesinger, 2016-06-24 22:22:31]

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

[commit-bot: I haz the power, 2016-06-24 22:22:56]

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

[cbiesinger, 2016-06-24 22:23:00]

The CQ bit was unchecked by cbiesinger@chromium.org

[cbiesinger, 2016-06-27 18:32:02]

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

[commit-bot: I haz the power, 2016-06-27 18:33:21]

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

[cbiesinger, 2016-06-27 18:40:39]

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

[commit-bot: I haz the power, 2016-06-27 18:41:12]

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

[cbiesinger, 2016-06-27 18:42:18]

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

[commit-bot: I haz the power, 2016-06-27 18:42:53]

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-06-27 18:55:49]

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

[commit-bot: I haz the power, 2016-06-27 18:55:50]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[cbiesinger, 2016-06-27 19:31:37]

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

[commit-bot: I haz the power, 2016-06-27 19:32:56]

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

[cbiesinger, 2016-06-27 19:48:42]

cbiesinger@chromium.org changed reviewers:
+ dgrogan@chromium.org, eae@chromium.org

[cbiesinger, 2016-06-27 19:49:49]

Note, this patch also renames availableFreeSpace to initialFreeSpace to be
clearer and to match the wording in the flexbox spec.

[eae, 2016-06-27 20:51:51]

LGTM

[cbiesinger, 2016-06-27 20:54:03]

Description was changed from

==========
[css-flexbox] Correctly implement freezing of inflexible items

We did not have a correct implementation of steps 2 and 3 of the
"9.7 Resolving flexible lengths" section of the spec:
https://drafts.csswg.org/css-flexbox/#resolve-flexible-lengths

In particular inflexible items (flex factor 0 or flex basis
violating the min/max size constraint) reduce the initial free
space.

This is an issue when sizing flex items with a flex factor of 0..1.

BUG=617956
==========

to

==========
[css-flexbox] Correctly implement freezing of inflexible items

We did not have a correct implementation of steps 2 and 3 of the
"9.7 Resolving flexible lengths" section of the spec:
https://drafts.csswg.org/css-flexbox/#resolve-flexible-lengths

In particular inflexible items (flex factor 0 or flex basis
violating the min/max size constraint) reduce the initial free
space.

This is an issue when sizing flex items with a flex factor of 0..1.

BUG=617956
TEST=css3/flexbox/max-width-violation.html
==========

[cbiesinger, 2016-06-27 20:54:27]

The CQ bit was unchecked by cbiesinger@chromium.org

[cbiesinger, 2016-06-27 20:54:33]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2016-06-27 20:55: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-06-27 21:07:12]

Description was changed from

==========
[css-flexbox] Correctly implement freezing of inflexible items

We did not have a correct implementation of steps 2 and 3 of the
"9.7 Resolving flexible lengths" section of the spec:
https://drafts.csswg.org/css-flexbox/#resolve-flexible-lengths

In particular inflexible items (flex factor 0 or flex basis
violating the min/max size constraint) reduce the initial free
space.

This is an issue when sizing flex items with a flex factor of 0..1.

BUG=617956
TEST=css3/flexbox/max-width-violation.html
==========

to

==========
[css-flexbox] Correctly implement freezing of inflexible items

We did not have a correct implementation of steps 2 and 3 of the
"9.7 Resolving flexible lengths" section of the spec:
https://drafts.csswg.org/css-flexbox/#resolve-flexible-lengths

In particular inflexible items (flex factor 0 or flex basis
violating the min/max size constraint) reduce the initial free
space.

This is an issue when sizing flex items with a flex factor of 0..1.

BUG=617956
TEST=css3/flexbox/max-width-violation.html
==========

[commit-bot: I haz the power, 2016-06-27 21:07:13]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-06-27 21:09:34]

Description was changed from

==========
[css-flexbox] Correctly implement freezing of inflexible items

We did not have a correct implementation of steps 2 and 3 of the
"9.7 Resolving flexible lengths" section of the spec:
https://drafts.csswg.org/css-flexbox/#resolve-flexible-lengths

In particular inflexible items (flex factor 0 or flex basis
violating the min/max size constraint) reduce the initial free
space.

This is an issue when sizing flex items with a flex factor of 0..1.

BUG=617956
TEST=css3/flexbox/max-width-violation.html
==========

to

==========
[css-flexbox] Correctly implement freezing of inflexible items

We did not have a correct implementation of steps 2 and 3 of the
"9.7 Resolving flexible lengths" section of the spec:
https://drafts.csswg.org/css-flexbox/#resolve-flexible-lengths

In particular inflexible items (flex factor 0 or flex basis
violating the min/max size constraint) reduce the initial free
space.

This is an issue when sizing flex items with a flex factor of 0..1.

BUG=617956
TEST=css3/flexbox/max-width-violation.html

Committed: https://crrev.com/143408653a5c0328254ea06d0f3a825d6505a114
Cr-Commit-Position: refs/heads/master@{#402279}
==========

[commit-bot: I haz the power, 2016-06-27 21:09:36]

Patchset 6 (id:??) landed as
https://crrev.com/143408653a5c0328254ea06d0f3a825d6505a114
Cr-Commit-Position: refs/heads/master@{#402279}