Replaced elements also need to check for flexbox when computing height

Replaced elements also need to check for flexbox when computing height

LayoutReplaced uses hasAutoHeightOrContainingBlockWithAutoHeight() to see
to use the specified height. But Flexbox adds some cases where heights should
be considered definite, so this function needs to check for it too.

(computePercentageLogicalHeight already has the check. Longer-term, we
should consider whether hasAutoHeightOrContainingBlockWithAutoHeight can
be implemented using computePercentageLogicalHeight)

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

[cbiesinger, 2016-06-16 13:33:59]

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

[commit-bot: I haz the power, 2016-06-16 13:34:08]

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

[eae, 2016-06-16 14:56:36]

LGTM

Nice test

[cbiesinger, 2016-06-16 15:04:50]

The CQ bit was unchecked by cbiesinger@chromium.org

[cbiesinger, 2016-06-16 15:04:59]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2016-06-16 15:05:25]

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

[commit-bot: I haz the power, 2016-06-16 15:10:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-16 15:10:33]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-16 15:12:05]

Description was changed from

==========
Replaced elements also need to check for flexbox when computing height

LayoutReplaced uses hasAutoHeightOrContainingBlockWithAutoHeight() to see
to use the specified height. But Flexbox adds some cases where heights should
be considered definite, so this function needs to check for it too.

(computePercentageLogicalHeight already has the check. Longer-term, we
should consider whether hasAutoHeightOrContainingBlockWithAutoHeight can
be implemented using computePercentageLogicalHeight)

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

to

==========
Replaced elements also need to check for flexbox when computing height

LayoutReplaced uses hasAutoHeightOrContainingBlockWithAutoHeight() to see
to use the specified height. But Flexbox adds some cases where heights should
be considered definite, so this function needs to check for it too.

(computePercentageLogicalHeight already has the check. Longer-term, we
should consider whether hasAutoHeightOrContainingBlockWithAutoHeight can
be implemented using computePercentageLogicalHeight)

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

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

[commit-bot: I haz the power, 2016-06-16 15:12:07]

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