Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1e2d07153e5938d1e318a46472ad025252756010
      
https://github.com/WebKit/WebKit/commit/1e2d07153e5938d1e318a46472ad025252756010
  Author: Ahmad Saleem <[email protected]>
  Date:   2026-06-08 (Mon, 08 Jun 2026)

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

  Log Message:
  -----------
  Use default member initializers for Table Layout data members
https://bugs.webkit.org/show_bug.cgi?id=316589
rdar://179044738

Reviewed by Alan Baradlay.

Follow-up to 314742@main, applying the same cleanup to the rest of the
table renderers. RenderTableCell and RenderTableRow each have two
constructors (Element& and Document&) that duplicated their member-init
lists; move those members to default member initializers in the header so
the constructors collapse to the base call. AutoTableLayout's single
constructor is likewise emptied.

The unsetColumnIndex/unsetRowIndex sentinels and the non-default
m_effectiveLogicalWidthDirty { true } are preserved. RenderTableCell's
seven 1-bit flags are also switched from unsigned to bool now that the
MSVC-only mixed-type packing workaround is no longer needed (since no port
uses MSVC compiler - drive-by fix).

No change in behavior.

* Source/WebCore/rendering/AutoTableLayout.cpp:
(WebCore::AutoTableLayout::AutoTableLayout):
* Source/WebCore/rendering/AutoTableLayout.h:
* Source/WebCore/rendering/RenderTableCell.cpp:
(WebCore::RenderTableCell::RenderTableCell):
* Source/WebCore/rendering/RenderTableCell.h:
* Source/WebCore/rendering/RenderTableRow.cpp:
(WebCore::RenderTableRow::RenderTableRow):
(WebCore::m_rowIndex): Deleted.
* Source/WebCore/rendering/RenderTableRow.h:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to