Skip to content

Conversation

@smfr
Copy link
Contributor

@smfr smfr commented Feb 1, 2023

1b555d0

Shrink Layout::ElementBox slightly
https://bugs.webkit.org/show_bug.cgi?id=251475
rdar://104894626

Reviewed by Alan Baradlay.

Shrink Layout::ElementBox from 184 bytes (13 bytes of padding) to 176 bytes
(5 bytes of padding) by shifting the bitfields in Box up into the spare
4 bytes between CanMakeCheckedPtr's m_count and m_style.

* Source/WebCore/layout/layouttree/LayoutBox.cpp:
(WebCore::Layout::Box::Box):
* Source/WebCore/layout/layouttree/LayoutBox.h:
* Source/WebCore/layout/layouttree/LayoutElementBox.h:
* Source/WebCore/rendering/style/RenderStyle.cpp:

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

6bbd39c

Misc iOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 🧪 win
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🛠 gtk 🛠 wincairo
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 gtk-wk2
✅ 🧪 api-ios ✅ 🧪 mac-wk1 🧪 api-gtk
✅ 🛠 tv ✅ 🧪 mac-wk2
✅ 🛠 tv-sim 🧪 mac-AS-debug-wk2
✅ 🛠 watch ✅ 🧪 mac-wk2-stress
✅ 🛠 🧪 merge ✅ 🛠 watch-sim

@smfr smfr self-assigned this Feb 1, 2023
@smfr smfr added the Layout and Rendering For bugs with layout and rendering of Web pages. label Feb 1, 2023
Comment on lines +93 to +95
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

@smfr smfr added the merge-queue Applied to send a pull request to merge-queue label Feb 1, 2023
https://bugs.webkit.org/show_bug.cgi?id=251475
rdar://104894626

Reviewed by Alan Baradlay.

Shrink Layout::ElementBox from 184 bytes (13 bytes of padding) to 176 bytes
(5 bytes of padding) by shifting the bitfields in Box up into the spare
4 bytes between CanMakeCheckedPtr's m_count and m_style.

* Source/WebCore/layout/layouttree/LayoutBox.cpp:
(WebCore::Layout::Box::Box):
* Source/WebCore/layout/layouttree/LayoutBox.h:
* Source/WebCore/layout/layouttree/LayoutElementBox.h:
* Source/WebCore/rendering/style/RenderStyle.cpp:

Canonical link: https://commits.webkit.org/259688@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/Shrink-LayoutElementBox-slightly branch from 6bbd39c to 1b555d0 Compare February 1, 2023 16:46
@webkit-commit-queue
Copy link
Collaborator

Committed 259688@main (1b555d0): https://commits.webkit.org/259688@main

Reviewed commits have been landed. Closing PR #9424 and removing active labels.

@webkit-early-warning-system webkit-early-warning-system merged commit 1b555d0 into WebKit:main Feb 1, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Layout and Rendering For bugs with layout and rendering of Web pages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants