Bug 1717956 [wpt PR 29469] - Fix under-invalidation of collapsed bord…

…er on removal of a table cell, a=testonly Automatic update from web-platform-tests Fix under-invalidation of collapsed border on removal of a table cell Collapsed borders need repaint when grid structure changes. We need to invalidate the table which paints collapsed borders. Bug: 1221182 Change-Id: I82d38f042602d134e9c148a997d3e06b76668074 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2983733 Commit-Queue: Xianzhu Wang <[email protected]> Reviewed-by: Aleks Totic <[email protected]> Cr-Commit-Position: refs/heads/master@{#895774} -- wpt-commits: 1ab2113bafa2171064dc20127fbc3fea984bc98d wpt-pr: 29469
jamienicol · Jun 26, 2021 · 96e1f9b · 96e1f9b
1 parent 3f63c0e
commit 96e1f9b
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 0 deletions.
diff --git a/testing/web-platform/tests/css/css-tables/collapsed-border-remove-cell-ref.html b/testing/web-platform/tests/css/css-tables/collapsed-border-remove-cell-ref.html
@@ -0,0 +1,18 @@
+<!DOCTYPE html>
+<style>
+  table {
+    border-collapse: collapse;
+  }
+  td {
+    width: 100px;
+    height: 100px;
+    border: 1px solid black;
+  }
+  td#target {
+    border-style: none;
+  }
+</style>
+<table>
+  <tr><td></td><td></td></tr>
+  <tr><td></td><td id="target"></tr>
+</table>
diff --git a/testing/web-platform/tests/css/css-tables/collapsed-border-remove-cell.html b/testing/web-platform/tests/css/css-tables/collapsed-border-remove-cell.html
@@ -0,0 +1,27 @@
+<!DOCTYPE html>
+<html class="reftest-wait">
+<title>Collapsed borders track changes caused by removal of cells</title>
+<link rel="help" href="https://drafts.csswg.org/css-tables/#border-collapsing">
+<link rel="match" href="collapsed-border-remove-cell-ref.html">
+<script src="/common/reftest-wait.js"></script>
+<script src="/common/rendering-utils.js"></script>
+<style>
+  table {
+    border-collapse: collapse;
+  }
+  td {
+    width: 100px;
+    height: 100px;
+    border: 1px solid black;
+  }
+</style>
+<table>
+  <tr><td></td><td></td></tr>
+  <tr><td></td><td id="target"></tr>
+</table>
+<script>
+  waitForAtLeastOneFrame().then(() => {
+    target.remove();
+    takeScreenshot();
+  });
+</script>