Let floats to use all available size if opportunity is infinite

Let floats to use all available size if opportunity is infinite

Because of the implementation specific of the layout opportunity iterator
an empty opportunity can mean 2 things:
- search for layout opportunities is exhausted.
- opportunity has an infinite size. That's because CS is infinite.

This patch fixes DCHECK crashes when we return an infinite opportunity.

BUG=635619
Review-Url: https://codereview.chromium.org/2801053004
Cr-Commit-Position: refs/heads/master@{#462969}
Committed: https://chromium.googlesource.com/chromium/src/+/958c1c976cd22ec4ed8e5695351ea3868b096961

[Gleb Lanbin, 2017-04-07 00:02:26]

Patchset #1 (id:1) has been deleted

[Gleb Lanbin, 2017-04-07 00:02:45]

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

[Gleb Lanbin, 2017-04-07 00:04:30]

Description was changed from

==========
Let float to use all available size if opportunity is infinite

Because of the implementation specific of the layout opportunity iterator
an empty opportunity can mean 2 things:
- search for layout opportunities is exhausted.
- opportunity has an infinite size. That's because CS is infinite.

This patch fixes DCHECK crashes when we return an infinite opportunity.

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

to

==========
Let floats to use all available size if opportunity is infinite

Because of the implementation specific of the layout opportunity iterator
an empty opportunity can mean 2 things:
- search for layout opportunities is exhausted.
- opportunity has an infinite size. That's because CS is infinite.

This patch fixes DCHECK crashes when we return an infinite opportunity.

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

[ikilpatrick, 2017-04-07 17:12:33]

lgtm

[Gleb Lanbin, 2017-04-07 17:14:25]

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

[commit-bot: I haz the power, 2017-04-07 17:15:14]

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-07 19:19:34]

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

[commit-bot: I haz the power, 2017-04-07 19:19:34]

Dry run: This issue passed the CQ dry run.

[Gleb Lanbin, 2017-04-07 19:33:41]

The CQ bit was checked by glebl@chromium.org

[commit-bot: I haz the power, 2017-04-07 19:34:13]

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-07 19:46:02]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491593621554950,
"parent_rev": "dbce02f63421f7d446d6ef317b456f6ae911f63b", "commit_rev":
"958c1c976cd22ec4ed8e5695351ea3868b096961"}

[commit-bot: I haz the power, 2017-04-07 19:47:21]

Description was changed from

==========
Let floats to use all available size if opportunity is infinite

Because of the implementation specific of the layout opportunity iterator
an empty opportunity can mean 2 things:
- search for layout opportunities is exhausted.
- opportunity has an infinite size. That's because CS is infinite.

This patch fixes DCHECK crashes when we return an infinite opportunity.

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

to

==========
Let floats to use all available size if opportunity is infinite

Because of the implementation specific of the layout opportunity iterator
an empty opportunity can mean 2 things:
- search for layout opportunities is exhausted.
- opportunity has an infinite size. That's because CS is infinite.

This patch fixes DCHECK crashes when we return an infinite opportunity.

BUG=635619

Review-Url: https://codereview.chromium.org/2801053004
Cr-Commit-Position: refs/heads/master@{#462969}
Committed:
https://chromium.googlesource.com/chromium/src/+/958c1c976cd22ec4ed8e5695351e...
==========

[commit-bot: I haz the power, 2017-04-07 19:47:23]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/958c1c976cd22ec4ed8e5695351e...