tree cf0b4701d5585a0464931216805ff54c87f26ce5
parent b40888afa9ab1d3e64c7085f4cf8972f56bfce1d
author Andreu Botella <abotella@igalia.com> 1698741495 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1698742656 -0700

Paint background invalidation when a table column span changes

Although table column elements (`LayoutTableColumn`) are not laid out,
their background styles will affect the background of the table
columns they span. The set of table columns that a table column
element spans can change, which means the background of the table
might need to change when a `LayoutTableColumn`'s span changes.

However, this is not currently happening because changing the span is
not invalidating the table background. This patch fixes that. As an
optimization, we invalidate the background only if some of the table's
`LayoutTableColumn` have a CSS-specified background.

For this optimization, however, `LayoutTable::HasBackgroundForPaint`
cannot be used, because that will only take into account the
backgrounds of any `LayoutTableColumn`s that span actual columns. It
is possible for a `LayoutTableColumn` that did not span any table
columns to suddenly do so if the span for some previous
`LayoutTableColumn` is decreased. Therefore, we instead added an
anonymous `TableHasColumnsWithBackground` function that checks every
`LayoutTableColumn`.

Bug: 1373252
Change-Id: I05edf22e5709ce33c574dfbe5f4f1c0da62f3a38
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4974436
Commit-Queue: Andreu Botella <abotella@igalia.com>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1217453}