Fix min-content sizing for 'word-break: break-word'

Fix min-content sizing for 'word-break: break-word'

This patch fixes min-content[1] sizing for 'word-break: break-word'.

The fix for crbug.com/655590 computes min-content sizing for
'break-word' the same as 'break-all', even though these two values have
different break opportunities as per the spec[2].

Due to this, when 'break-word' is applied to text which 'break-all' does
not allow break, such as a long sequence of '.' (FULL STOP, a.k.a.
period,) the min-content becomes too long.

[1] https://drafts.csswg.org/css-sizing/#min-content
[2] https://drafts.csswg.org/css-text-3/#word-break-property

BUG=687338
Review-Url: https://codereview.chromium.org/2681633002
Cr-Commit-Position: refs/heads/master@{#448744}
Committed: https://chromium.googlesource.com/chromium/src/+/eb5a8c59e9f3121d5c0dc92b2539f86f847eb295

[kojii, 2017-02-07 02:11:49]

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

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

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kojii, 2017-02-07 02:19:59]

Description was changed from

==========
WIP: break-all

BUG=687338
==========

to

==========
Fix min-content sizing for 'word-break: break-word'

This patch fixes min-content[1] sizing for 'word-break: break-word'.

The fix for crbug.com/655590 computes min-content sizing for
'break-word' the same as 'break-all', even though these two values have
different break opportunities as per the spec[2].

Due to this, when 'break-word' is applied to text which 'break-all' does
not allow break, such as a long sequence of '.' (FULL STOP, a.k.a.
period,) the min-content becomes too long.

[1] https://drafts.csswg.org/css-sizing/#min-content
[2] https://drafts.csswg.org/css-text-3/#word-break-property

BUG=687338
==========

[commit-bot: I haz the power, 2017-02-07 04:10:30]

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

[commit-bot: I haz the power, 2017-02-07 04:10:31]

Dry run: This issue passed the CQ dry run.

[kojii, 2017-02-07 04:52:57]

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

[kojii, 2017-02-07 04:52:57]

PTAL.

[eae, 2017-02-07 17:40:57]

LGTM

[kojii, 2017-02-07 22:10:52]

The CQ bit was checked by kojii@chromium.org

[commit-bot: I haz the power, 2017-02-07 22:11:59]

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-02-07 22:27:23]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486505452841660, "parent_rev":
"e5f19f64a46e9d607192697105b7db46f79e298c", "commit_rev":
"eb5a8c59e9f3121d5c0dc92b2539f86f847eb295"}

[commit-bot: I haz the power, 2017-02-07 22:27:59]

Description was changed from

==========
Fix min-content sizing for 'word-break: break-word'

This patch fixes min-content[1] sizing for 'word-break: break-word'.

The fix for crbug.com/655590 computes min-content sizing for
'break-word' the same as 'break-all', even though these two values have
different break opportunities as per the spec[2].

Due to this, when 'break-word' is applied to text which 'break-all' does
not allow break, such as a long sequence of '.' (FULL STOP, a.k.a.
period,) the min-content becomes too long.

[1] https://drafts.csswg.org/css-sizing/#min-content
[2] https://drafts.csswg.org/css-text-3/#word-break-property

BUG=687338
==========

to

==========
Fix min-content sizing for 'word-break: break-word'

This patch fixes min-content[1] sizing for 'word-break: break-word'.

The fix for crbug.com/655590 computes min-content sizing for
'break-word' the same as 'break-all', even though these two values have
different break opportunities as per the spec[2].

Due to this, when 'break-word' is applied to text which 'break-all' does
not allow break, such as a long sequence of '.' (FULL STOP, a.k.a.
period,) the min-content becomes too long.

[1] https://drafts.csswg.org/css-sizing/#min-content
[2] https://drafts.csswg.org/css-text-3/#word-break-property

BUG=687338

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

[commit-bot: I haz the power, 2017-02-07 22:28:01]

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