1531357 - [wpt-sync] Sync PR 15509 - [LayoutNG] Fix 3 cases of break opportunities after nowrap

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 15509 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/15509
Details from upstream follow.

Koji Ishii <[e-mail]> wrote:

[LayoutNG] Fix 3 cases of break opportunities after nowrap

This patch fixes lines to break in the following conditions:

1. When wrappable elements appear inside of nowrap elements.
2. When wrappable spaces after nowrap appear inside of nowrap
   elements.
3. When non-space break opportunities appear after nowrap.

fast/text/whitespace/018.html improves but still doesn't pass.
It doesn't pass in Edge/Gecko, and at least some of what it
expects look questionable. Further investigation is deferred
to future CLs.

Bug: 920177
Change-Id: Ieba4d446b818120f423b87a7f4a44b3c63a9d995
Reviewed-on: https://chromium-review.googlesource.com/c/1477629
Commit-Queue: Koji Ishii \<[e-mail]>
Reviewed-by: Xiaocheng Hu \<[e-mail]>
Reviewed-by: Emil A Eklund \<[e-mail]>
Cr-Commit-Position: refs/heads/master@{#635180}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=b81df24c7402c9aed43e08c266f9fef533bc38c9

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Ran 1 tests
FAIL   : 1

New tests that have failures or other problems:
/css/css-text/white-space/white-space-wrap-after-nowrap-001.html: FAIL

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3f555f93c070
[wpt PR 15509] - [LayoutNG] Fix 3 cases of break opportunities after nowrap, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/7b01e9d570db
[wpt PR 15509] - Update wpt metadata, a=testonly

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/272d41054b82
[wpt PR 15509] - [LayoutNG] Fix 3 cases of break opportunities after nowrap, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/45cdd214896f
[wpt PR 15509] - Update wpt metadata, a=testonly

Comment 5

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/272d41054b82
https://hg.mozilla.org/mozilla-central/rev/45cdd214896f