Bug 1692179 [wpt PR 27590] - Revert "[TablesNG] Implement column track merging",...
commitf573ba9a637ab6e758974083f82b19e34b717951
authorThiemo Nagel <tnagel@chromium.org>
Thu, 11 Feb 2021 16:50:24 +0000 (11 16:50 +0000)
committermoz-wptsync-bot <wptsync@mozilla.com>
Fri, 12 Feb 2021 12:12:41 +0000 (12 12:12 +0000)
tree88da5419556dbd34a5e679c176670a0af302b1f7
parentc8a8f29284484e7b368702ff944e3a3464f3fe3d
Bug 1692179 [wpt PR 27590] - Revert "[TablesNG] Implement column track merging", a=testonly

Automatic update from web-platform-tests
Revert "[TablesNG] Implement column track merging"

This reverts commit 2dadda0cccbda46a2fffe23e8c607e88f4d45394.

Reason for revert: Suspected cause of paint/tables/huge-table-composited-scroll-collapsed-borders.html failures

https://ci.chromium.org/ui/p/chromium/builders/ci/WebKit%20Linux%20ASAN/22054

Original change's description:
> [TablesNG] Implement column track merging
>
> One the a11y requirements was that tables should do track merging.
> I've skipped implementing this initially because there was major
> disagreement about what to do.
>
> This implementation follows the standard.
> Columns defined by TD or COL are merged for table-layout:auto,
> and not merged for table-layout:fixed.
>
> Legacy did not follow the standard for columns defined by TDs:
> they were always merged. Firefox implementation was closer to
> standard, but sometimes buggy.
>
> This changes results of some tests. Almost all of them are because
> "TablesNG is more standards compliant", and new renderings match
> FF.
>
> There is disagreement about treatment of empty rows between
> FF and TablesNGNG.
>
> external/wpt/css/css-tables/html5-table-formatting-1.html
> external/wpt/css/css-tables/html5-table-formatting-2.html
>
> The rest of the changes render better, just like FF.
>
> Two new tests have been added:
> wpt/css/css-tables/column-track-merging.html
> tests the standard.
>
> accessibility/table-column-track-merging.html
> tests Chrome's a11y API.
>
> Change-Id: Ia222351f7c500963ace556d76c26fcefb1df24e1
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2670193
> Commit-Queue: Aleks Totic <atotic@chromium.org>
> Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
> Reviewed-by: David Grogan <dgrogan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#852958}

TBR=dgrogan@chromium.org,ikilpatrick@chromium.org,atotic@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com

Change-Id: Id914684949be4eb6196a850bb1168d25872346ef
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2690334
Reviewed-by: Thiemo Nagel <tnagel@chromium.org>
Commit-Queue: Thiemo Nagel <tnagel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#853075}

--

wpt-commits: 496c290543d0572ec532b092a3b2e7fe03381293
wpt-pr: 27590
testing/web-platform/tests/css/css-tables/column-track-merging.html [deleted file]