[LayoutNG] Unit tests for MinAndMaxContentSizes::ShrinkToFit().

[LayoutNG] Unit tests for MinAndMaxContentSizes::ShrinkToFit().

Also DCHECK in the implementation that max_content isn't less than min_content.

Committed: https://crrev.com/c255d0b0d14ccd32c7c88b009cbd0c94291fa3e1
Cr-Commit-Position: refs/heads/master@{#434458}

[mstensho (USE GERRIT), 2016-11-23 21:16:44]

Description was changed from

==========
Unit tests for MinAndMaxContentSizes::ShrinkToFit().

Also DCHECK in the implementation that max_content isn't less than min_content.
==========

to

==========
[LayoutNG] Unit tests for MinAndMaxContentSizes::ShrinkToFit().

Also DCHECK in the implementation that max_content isn't less than min_content.
==========

[mstensho (USE GERRIT), 2016-11-23 21:19:48]

mstensho@opera.com changed reviewers:
+ atotic@chromium.org

[mstensho (USE GERRIT), 2016-11-23 21:19:49]

First I actually thought that the shrink-to-fit algorithm should say:

max(min_content, min(max_content, available_space))

So I got curious. Turns out that what we already have here is exactly what the
spec says. It doesn't really matter which one we use, as long as max_content >=
min_content, though.

[atotic, 2016-11-24 23:19:17]

On 2016/11/23 at 21:19:49, mstensho wrote:
> First I actually thought that the shrink-to-fit algorithm should say:
> 
> max(min_content, min(max_content, available_space))
> 
> So I got curious. Turns out that what we already have here is exactly what the
spec says. It doesn't really matter which one we use, as long as max_content >=
min_content, though.

lgtm

[mstensho (USE GERRIT), 2016-11-25 07:17:20]

The CQ bit was checked by mstensho@opera.com

[commit-bot: I haz the power, 2016-11-25 07:17:38]

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-11-25 08:53:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480058240768990, "parent_rev":
"831ac2666c116bf6dd30da3c06f18d0dd739873c", "commit_rev":
"b5e6c86f8a5ce6513b96bb8f51729a2b0fc683ea"}

[commit-bot: I haz the power, 2016-11-25 08:53:43]

Description was changed from

==========
[LayoutNG] Unit tests for MinAndMaxContentSizes::ShrinkToFit().

Also DCHECK in the implementation that max_content isn't less than min_content.
==========

to

==========
[LayoutNG] Unit tests for MinAndMaxContentSizes::ShrinkToFit().

Also DCHECK in the implementation that max_content isn't less than min_content.
==========

[commit-bot: I haz the power, 2016-11-25 08:53:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-25 08:55:47]

Description was changed from

==========
[LayoutNG] Unit tests for MinAndMaxContentSizes::ShrinkToFit().

Also DCHECK in the implementation that max_content isn't less than min_content.
==========

to

==========
[LayoutNG] Unit tests for MinAndMaxContentSizes::ShrinkToFit().

Also DCHECK in the implementation that max_content isn't less than min_content.

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

[commit-bot: I haz the power, 2016-11-25 08:55:48]

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