Don't let anonymous boxes impede percentage resolution on children.

Don't let anonymous boxes impede percentage resolution on children.

We already had code in place to do this for certain anonymous box types, most
prominently multicol flow threads. But no type of anonymous box should really
mess up percentage resolution on children. It's just wrong, and in this case it
even caused assertion failures, because anonymous ruby run boxes could end up
being marked as percentage height parents. Since what child belongs inside what
ruby run may change depending on where the ruby texts are inserted, and we have
no code to adjust the percentage height map when percentage height children get
moved from one ruby run to another (and so far there's nothing that suggests
that we really need it either), let's just avoid the whole problem by skipping
ruby runs when resolving percentage heights - with the bonus effect that
percentage-height children of ruby elements actually get rendered properly.

Some cleanup was natural at the same time, since we now bail early for
any type of anonymous box in skipContainingBlockForPercentHeightCalculation().

BUG=490722
R=leviw@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196713

[leviw_travelin_and_unemployed, 2015-06-08 22:47:05]

A patch with a *bonus* effect!? LGTM! :)

[leviw_travelin_and_unemployed, 2015-06-08 22:47:07]

The CQ bit was checked by leviw@chromium.org

[commit-bot: I haz the power, 2015-06-08 22:47:45]

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

[commit-bot: I haz the power, 2015-06-08 23:45:51]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196713