[LayoutNG] Return MinAndMaxContentSizes by value, step 1

[LayoutNG] Return MinAndMaxContentSizes by value, step 1

It's small enough that we probably don't need to worry about copying it.

ComputeMinAndMaxContentSizesSync was a small convenience wrapper that
did this; we don't need it anymore.

Step 2 will tackle NGBlockLayoutAlgorithm::ComputeMinAndMaxContentSizes

BUG=635619
Review-Url: https://codereview.chromium.org/2656693007
Cr-Commit-Position: refs/heads/master@{#446421}
Committed: https://chromium.googlesource.com/chromium/src/+/2f09779466558e6e2ce51f9fa74ec00c10cdc8c6

[dgrogan, 2017-01-25 23:31:38]

dgrogan@chromium.org changed reviewers:
+ glebl@chromium.org

[dgrogan, 2017-01-25 23:31:39]

ptal?

[Gleb Lanbin, 2017-01-25 23:49:28]

lgtm

[ikilpatrick, 2017-01-26 04:39:09]

ikilpatrick@chromium.org changed reviewers:
+ ikilpatrick@chromium.org

[ikilpatrick, 2017-01-26 04:39:10]

lgtm

[ikilpatrick, 2017-01-26 04:39:21]

thanks :)

[dgrogan, 2017-01-26 16:52:52]

The CQ bit was checked by dgrogan@chromium.org

[commit-bot: I haz the power, 2017-01-26 16:53:07]

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-01-26 20:10:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485449572906320, "parent_rev":
"2476c81a93b0cb04c10c8186308fbb5cf1271627", "commit_rev":
"2f09779466558e6e2ce51f9fa74ec00c10cdc8c6"}

[commit-bot: I haz the power, 2017-01-26 20:11:27]

Description was changed from

==========
[LayoutNG] Return MinAndMaxContentSizes by value, step 1

It's small enough that we probably don't need to worry about copying it.

ComputeMinAndMaxContentSizesSync was a small convenience wrapper that
did this; we don't need it anymore.

Step 2 will tackle NGBlockLayoutAlgorithm::ComputeMinAndMaxContentSizes

BUG=635619
==========

to

==========
[LayoutNG] Return MinAndMaxContentSizes by value, step 1

It's small enough that we probably don't need to worry about copying it.

ComputeMinAndMaxContentSizesSync was a small convenience wrapper that
did this; we don't need it anymore.

Step 2 will tackle NGBlockLayoutAlgorithm::ComputeMinAndMaxContentSizes

BUG=635619

Review-Url: https://codereview.chromium.org/2656693007
Cr-Commit-Position: refs/heads/master@{#446421}
Committed:
https://chromium.googlesource.com/chromium/src/+/2f09779466558e6e2ce51f9fa74e...
==========

[commit-bot: I haz the power, 2017-01-26 20:11:35]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2f09779466558e6e2ce51f9fa74e...

[cbiesinger, 2017-01-31 01:04:44]

lgtm