1497319 - [wpt-sync] Sync PR 13359 - [LayoutNG] Correct LayoutText::Quads() for vertical-rl.

Status: RESOLVED FIXED

(Core :: DOM: CSS Object Model, enhancement, P4)

Description

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

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

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

Morten Stenshorne <[e-mail]> wrote:
>  [LayoutNG] Correct LayoutText::Quads() for vertical-rl.
>  
>  NG needs to produce a rectangle with the block-axis offset relatively to
>  block-start of the container, just like legacy does. Using purely
>  physical coordinates is wrong.
>  
>  Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
>  Change-Id: I9989628dcc339c71d850818498538aafa142ed29
>  Reviewed-on: https://chromium-review.googlesource.com/c/1261698
>  Reviewed-by: Koji Ishii \<[e-mail]>
>  Commit-Queue: Morten Stenshorne \<[e-mail]>
>  Cr-Commit-Position: refs/heads/master@{#596747}
>

Comment 1

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

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=e9ebf761bd6e2967dd355c3e40a465c57b1a1b6b

Comment 2

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

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

Comment 3

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

Ran 6 tests and 6 subtests
OK     : 6
PASS   : 3
FAIL   : 3

New tests that have failures or other problems:
/css/cssom-view/getClientRects-br-htb-rtl.html
    Position of the BR element: FAIL
/css/cssom-view/getClientRects-br-vlr-rtl.html
    Position of the BR element: FAIL
/css/cssom-view/getClientRects-br-vrl-rtl.html
    Position of the BR element: FAIL

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7c524ed4f7f2
[wpt PR 13359] - [LayoutNG] Correct LayoutText::Quads() for vertical-rl., a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/3fc8243dfd23
[wpt PR 13359] - Update wpt metadata, a=testonly

Comment 5

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/7c524ed4f7f2
https://hg.mozilla.org/mozilla-central/rev/3fc8243dfd23