[wpt-sync] Sync PR 15509 - [LayoutNG] Fix 3 cases of break opportunities after nowrap
Categories
(Core :: CSS Parsing and Computation, enhancement, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
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 <kojii@chromium.org> wrote:
[LayoutNG] Fix 3 cases of break opportunities after nowrap
This patch fixes lines to break in the following conditions:
- When wrappable elements appear inside of nowrap elements.
- When wrappable spaces after nowrap appear inside of nowrap
elements.- 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 \<kojii@chromium.org>
Reviewed-by: Xiaocheng Hu \<xiaochengh@chromium.org>
Reviewed-by: Emil A Eklund \<eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#635180}
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=b81df24c7402c9aed43e08c266f9fef533bc38c9
Assignee | ||
Comment 2•5 years ago
|
||
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
Pushed by james@hoppipolla.co.uk: 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
Pushed by james@hoppipolla.co.uk: 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•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/272d41054b82
https://hg.mozilla.org/mozilla-central/rev/45cdd214896f
Description
•