Treat cells with percent height as auto if their container depends on content height

Treat cells with percent height as auto if their container depends on content height

After some clarification with the css-tables spec author and the CSSWG, only
treat cells as having a height that percent values can resolve against if the cell
has an explicitly specified height. For example, if the cell has a percent height
but nothing in its container chain to resolve that percentage against it doesn't
have an explicitly specified height.

BUG=687551
Review-Url: https://codereview.chromium.org/2816313002
Cr-Commit-Position: refs/heads/master@{#466287}
Committed: https://chromium.googlesource.com/chromium/src/+/019c92b5d6c58f4c240ea2b64bb748c9bb3a6c15

[rhogan, 2017-04-15 17:27:07]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-15 17:27:12]

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, 2017-04-15 18:40:24]

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

[commit-bot: I haz the power, 2017-04-15 18:40:24]

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

[rhogan, 2017-04-16 09:56:42]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-16 09:56:46]

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, 2017-04-16 11:31:21]

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

[commit-bot: I haz the power, 2017-04-16 11:31:22]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[rhogan, 2017-04-19 22:04:05]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-19 22:04:58]

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, 2017-04-20 01:44:48]

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

[commit-bot: I haz the power, 2017-04-20 01:44:49]

Dry run: This issue passed the CQ dry run.

[rhogan, 2017-04-20 21:13:08]

robhogan@gmail.com changed reviewers:
+ eae@chromium.org

[rhogan, 2017-04-20 21:16:55]

Description was changed from

==========
Treat cells with percent height as auto if their container depends on content
height

BUG=687551
==========

to

==========
Treat cells with percent height as auto if their container depends on content
height

After some clarification with the css-tables spec author and the CSSWG, only
treat cells as having a height that percent values can resolve against if the
cell
has an explicitly specified height. For example, if the cell has a percent
height
but nothing in its container chain to resolve that percentage against it doesn't
have an explicitly specified height.

BUG=687551
==========

[eae, 2017-04-20 22:58:37]

LGTM

[rhogan, 2017-04-21 06:29:40]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2017-04-21 06:29:51]

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, 2017-04-21 08:03:42]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492756180108390,
"parent_rev": "7fd3f6718896969f1a5d3850a6d81f4ccf74abd3", "commit_rev":
"019c92b5d6c58f4c240ea2b64bb748c9bb3a6c15"}

[commit-bot: I haz the power, 2017-04-21 08:03:53]

Description was changed from

==========
Treat cells with percent height as auto if their container depends on content
height

After some clarification with the css-tables spec author and the CSSWG, only
treat cells as having a height that percent values can resolve against if the
cell
has an explicitly specified height. For example, if the cell has a percent
height
but nothing in its container chain to resolve that percentage against it doesn't
have an explicitly specified height.

BUG=687551
==========

to

==========
Treat cells with percent height as auto if their container depends on content
height

After some clarification with the css-tables spec author and the CSSWG, only
treat cells as having a height that percent values can resolve against if the
cell
has an explicitly specified height. For example, if the cell has a percent
height
but nothing in its container chain to resolve that percentage against it doesn't
have an explicitly specified height.

BUG=687551

Review-Url: https://codereview.chromium.org/2816313002
Cr-Commit-Position: refs/heads/master@{#466287}
Committed:
https://chromium.googlesource.com/chromium/src/+/019c92b5d6c58f4c240ea2b64bb7...
==========

[commit-bot: I haz the power, 2017-04-21 08:03:54]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/019c92b5d6c58f4c240ea2b64bb7...