[LayoutNG] Fix inline ComputeMinMaxContentSize()

[LayoutNG] Fix inline ComputeMinMaxContentSize()

This patch fixes two issues in NGInlineNode::ComputeMinMaxContentSize()

1. The max-content algorithm was incorrect. Fixed to compute the max of
   line widths when available width is infinite.
2. Negative text-indent can make min > max. CSSWG test indicates we
   should take min of the two values in the case.

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

Review-Url: https://codereview.chromium.org/2974823004
Cr-Commit-Position: refs/heads/master@{#485890}
Committed: https://chromium.googlesource.com/chromium/src/+/c8532b25d4af1355089002e5a7ba913409194d83

[kojii, 2017-07-11 07:52:13]

Description was changed from

==========
minmax

BUG=
==========

to

==========
minmax

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

[kojii, 2017-07-11 07:52:25]

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

[commit-bot: I haz the power, 2017-07-11 07:52:35]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kojii, 2017-07-11 07:57:31]

Description was changed from

==========
minmax

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Fix inline ComputeMinMaxContentSize()

This patch fixes two issues in NGInlineNode::ComputeMinMaxContentSize()

1. The max-content algorithm was incorrect. Fixed to compute the max of
   line widths when available width is infinite.
2. Negative text-indent can make min > max. CSSWG test indicates we
   should take min of the two values in the case.

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

[kojii, 2017-07-11 08:02:15]

kojii@chromium.org changed reviewers:
+ cbiesinger@chromium.org, eae@chromium.org

[kojii, 2017-07-11 08:02:16]

PTAL.

[commit-bot: I haz the power, 2017-07-11 08:47:14]

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

[commit-bot: I haz the power, 2017-07-11 08:47:15]

Dry run: Try jobs failed on following builders:
  linux_layout_tests_layout_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_layout_te...)

[ikilpatrick, 2017-07-11 16:55:01]

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

[ikilpatrick, 2017-07-11 16:55:01]

lgtm

[eae, 2017-07-11 22:21:04]

LGTM

[kojii, 2017-07-12 04:17:45]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2017-07-12 04:18:01]

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-07-12 05:28:54]

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

[commit-bot: I haz the power, 2017-07-12 05:28:56]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kojii, 2017-07-12 07:29:20]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2017-07-12 07:29:29]

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-07-12 07:43:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499844560676120, "parent_rev":
"f66aae2d690ff7d3d22bfd086a203bb5e9ce80a4", "commit_rev":
"c8532b25d4af1355089002e5a7ba913409194d83"}

[commit-bot: I haz the power, 2017-07-12 07:43:55]

Description was changed from

==========
[LayoutNG] Fix inline ComputeMinMaxContentSize()

This patch fixes two issues in NGInlineNode::ComputeMinMaxContentSize()

1. The max-content algorithm was incorrect. Fixed to compute the max of
   line widths when available width is infinite.
2. Negative text-indent can make min > max. CSSWG test indicates we
   should take min of the two values in the case.

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng
==========

to

==========
[LayoutNG] Fix inline ComputeMinMaxContentSize()

This patch fixes two issues in NGInlineNode::ComputeMinMaxContentSize()

1. The max-content algorithm was incorrect. Fixed to compute the max of
   line widths when available width is infinite.
2. Negative text-indent can make min > max. CSSWG test indicates we
   should take min of the two values in the case.

BUG=636993
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_layout_ng

Review-Url: https://codereview.chromium.org/2974823004
Cr-Commit-Position: refs/heads/master@{#485890}
Committed:
https://chromium.googlesource.com/chromium/src/+/c8532b25d4af1355089002e5a7ba...
==========

[commit-bot: I haz the power, 2017-07-12 07:43:57]

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