Closed Bug 1694763 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 27765 - Fix NG table collapsed border pixel-snapping and clipping

Categories

(Core :: Layout: Tables, task, P4)

task

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Xianzhu Wang <wangxianzhu@chromium.org>' wrote:

Fix NG table collapsed border pixel-snapping and clipping

This fixes the following issues:

  • We missed pixel-snapping of the collapsed borders.
    Add ObjectPainterBase::DrawBoxSide() which accepts IntRect instead of
    float parameters. Previously if the float parameters were not
    pixel-snapped, they would be converted to int by truncating which
    implicitly snapped to whole pixels incorrectly.

  • We missed pixel-snapping and shrinking by border outsets for the clip
    when painting background for layered table parts. Combine the clip
    logic for cell background and other table part background with
    correct pixel-snapping and clip rect, with a test.

Bug: 1180773
Change-Id: I81a967b3de08c116eed97a08e0f608a966088f40

Reviewed-on: https://chromium-review.googlesource.com/2717524
WPT-Export-Revision: dcc5d7946909547be5da9c6a29cdd732f6f0d272

Component: web-platform-tests → Layout: Tables
Product: Testing → Core

CI Results

Ran 15 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests

Status Summary

Firefox

FAIL: 1

Chrome

FAIL: 1

Safari

FAIL: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/css/css-tables/collapsed-border-positioned-tr-td.html: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8b6f898a24f2 [wpt PR 27765] - Fix NG table collapsed border pixel-snapping and clipping, a=testonly https://hg.mozilla.org/integration/autoland/rev/f71b1b1a662d [wpt PR 27765] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in before you can comment on or make changes to this bug.