[wpt-sync] Sync PR 19541 - [LayoutNG] Fix line breaking when `font-size: 0`
Categories
(Core :: CSS Parsing and Computation, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 19541 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/19541
Details from upstream follow.
Koji Ishii <kojii@chromium.org> wrote:
[LayoutNG] Fix line breaking when
font-size: 0
This patch fixes line breaking when break opportunities have
font-size: 0
.
font-size: 0
makes all characters at position 0. This
confuses when |NGLineBreaker| needs to rewind, and tries to
find the break opportunity before the current one.This patch fixes it by using |PreviousBreakOpportunity| in
such case.Bug: 1006127
Change-Id: I449d6f8aaba4deccd458c5e077920b1a8bf79305Reviewed-on: https://chromium-review.googlesource.com/1843991
WPT-Export-Revision: a11487f9fb95dcf4ee52257aca3bd2cb1e58ecfc
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=16789c919f698f295f2ebdd96ac8b39c786a5895
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=bc3e067ea065f7ca0dc1f110d1983db304a99eba
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=be948f7648f292f5a130f0e32a0ff5966c91f659
Assignee | ||
Comment 4•5 years ago
|
||
## GitHub CI Results wpt.fyi [PR Results](https://wpt.fyi/results/?sha=89ede65470ee76e543d98425da9da96b7be1b82a&label=pr_head) [Base Results](https://wpt.fyi/results/?sha=89ede65470ee76e543d98425da9da96b7be1b82a&label=pr_base) Ran 1 tests ### Firefox PASS: 1 ### Chrome FAIL: 1 ### Safari PASS: 1 ## Gecko CI Results Ran 1 tests PASS : 1
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c276749f30a6 [wpt PR 19541] - [LayoutNG] Fix line breaking when `font-size: 0`, a=testonly
Comment 6•5 years ago
|
||
bugherder |
Description
•