Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6204a23e455cf40f58a11ceab47a26683a28d4e9
      
https://github.com/WebKit/WebKit/commit/6204a23e455cf40f58a11ceab47a26683a28d4e9
  Author: Simon Fraser <[email protected]>
  Date:   2025-10-18 (Sat, 18 Oct 2025)

  Changed paths:
    M Source/WebCore/rendering/RenderTableCell.cpp
    M Source/WebCore/rendering/RenderTableRow.cpp
    M Source/WebCore/rendering/RenderTableSection.cpp

  Log Message:
  -----------
  Minor table painting optimizations
https://bugs.webkit.org/show_bug.cgi?id=301026
rdar://162912961

Reviewed by Alan Baradlay.

Remove the `setHasVisibleBoxDecorations(true()` from 
`RenderTableCell::styleDidChange()`.
This dates from the khtml days, and I don't believe it's necessary. 
`RenderBlock::paintObject()`
checks `hasVisibleBoxDecorations` before calling `paintBoxDecorations()`, and
`RenderTableCell::paintBoxDecorations()` only paints the box decorations for 
the cell, not for
the section, row, colgroup or col, so the check appears to be valid.

* Source/WebCore/rendering/RenderTableCell.cpp:
(WebCore::RenderTableCell::styleDidChange):
* Source/WebCore/rendering/RenderTableRow.cpp:
(WebCore::RenderTableRow::paintOutlineForRowIfNeeded): We only need to compute 
`adjustedPaintOffset` if calling `paintOutline`.
* Source/WebCore/rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::paintObject): Check `hasOutline()` before calling 
`row->paintOutlineForRowIfNeeded()`.

Canonical link: https://commits.webkit.org/301771@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to