[css-flexbox] Simplify the "child intrinsic height" logic

[css-flexbox] Simplify the "child intrinsic height" logic

The logic in childIntrinsicHeight/childIntrinsicWidth was very
hard to understand. This patch makes them work on the logical
width/height instead, which is much simpler.

This also removes a TODO mentioning indefinite percentages, because
the spec is clear that only actual auto values should be stretched.

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

Committed: https://crrev.com/73e356aec7507c9e27dcbd9728511e9775d03a44
Cr-Commit-Position: refs/heads/master@{#410348}

[cbiesinger, 2016-08-05 20:38:32]

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

[commit-bot: I haz the power, 2016-08-05 20:39:00]

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-05 20:39:55]

Description was changed from

==========
[css-flexbox] Simplify the "child intrinsic height" logic

The logic in childIntrinsicHeight/childIntrinsicWidth was very
hard to understand. This patch makes them work on the logical
width/height instead, which is much simpler.

This also removes a FIXME mentioning indefinite percentages, because
the spec is clear that only actual auto values should be stretched.

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

to

==========
[css-flexbox] Simplify the "child intrinsic height" logic

The logic in childIntrinsicHeight/childIntrinsicWidth was very
hard to understand. This patch makes them work on the logical
width/height instead, which is much simpler.

This also removes a TODO mentioning indefinite percentages, because
the spec is clear that only actual auto values should be stretched.

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

[commit-bot: I haz the power, 2016-08-05 20:50:22]

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

[commit-bot: I haz the power, 2016-08-05 20:50:23]

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

[cbiesinger, 2016-08-05 20:52:21]

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

[commit-bot: I haz the power, 2016-08-05 20:52:57]

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-05 21:35:54]

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

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

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-08-05 23:02:17]

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

[commit-bot: I haz the power, 2016-08-05 23:02:17]

Dry run: This issue passed the CQ dry run.

[eae, 2016-08-08 13:01:06]

Nice, LGTM

[cbiesinger1, 2016-08-08 13:07:38]

The CQ bit was checked by cbiesinger@google.com

[commit-bot: I haz the power, 2016-08-08 13:07:49]

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-08 14:20:10]

Description was changed from

==========
[css-flexbox] Simplify the "child intrinsic height" logic

The logic in childIntrinsicHeight/childIntrinsicWidth was very
hard to understand. This patch makes them work on the logical
width/height instead, which is much simpler.

This also removes a TODO mentioning indefinite percentages, because
the spec is clear that only actual auto values should be stretched.

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

to

==========
[css-flexbox] Simplify the "child intrinsic height" logic

The logic in childIntrinsicHeight/childIntrinsicWidth was very
hard to understand. This patch makes them work on the logical
width/height instead, which is much simpler.

This also removes a TODO mentioning indefinite percentages, because
the spec is clear that only actual auto values should be stretched.

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

[commit-bot: I haz the power, 2016-08-08 14:20:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-08 14:21:25]

Description was changed from

==========
[css-flexbox] Simplify the "child intrinsic height" logic

The logic in childIntrinsicHeight/childIntrinsicWidth was very
hard to understand. This patch makes them work on the logical
width/height instead, which is much simpler.

This also removes a TODO mentioning indefinite percentages, because
the spec is clear that only actual auto values should be stretched.

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

to

==========
[css-flexbox] Simplify the "child intrinsic height" logic

The logic in childIntrinsicHeight/childIntrinsicWidth was very
hard to understand. This patch makes them work on the logical
width/height instead, which is much simpler.

This also removes a TODO mentioning indefinite percentages, because
the spec is clear that only actual auto values should be stretched.

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

Committed: https://crrev.com/73e356aec7507c9e27dcbd9728511e9775d03a44
Cr-Commit-Position: refs/heads/master@{#410348}
==========

[commit-bot: I haz the power, 2016-08-08 14:21:28]

Patchset 3 (id:??) landed as
https://crrev.com/73e356aec7507c9e27dcbd9728511e9775d03a44
Cr-Commit-Position: refs/heads/master@{#410348}