[wpe-2.46] Fix build errors with malloc_heap_breakdown enabled

[nmuthu973]

f6f6f8e

Build-Tests	Layout-Tests
✅ 🛠 wpe-246-amd64-build	✅ 🧪 wpe-246-amd64-layout
✅ 🛠 wpe-246-arm32-build	✅ 🧪 wpe-246-arm32-layout

[Copilot]

Pull Request Overview

This PR adds missing header includes and removes an unnecessary allocator definition to fix compilation when ENABLE(MALLOC_HEAP_BREAKDOWN) is enabled.

• Added <wtf/NeverDestroyed.h> includes to files using DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER macro
• Added LoaderMalloc.h include to CachedSVGDocumentReference.h for heap identifier usage
• Removed redundant DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER from FrameRateAligner.cpp

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
Source/WebCore/rendering/style/StyleSurroundData.cpp	Adds NeverDestroyed.h include required by DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER macro
Source/WebCore/platform/graphics/Region.cpp	Adds NeverDestroyed.h include required by DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER macro
Source/WebCore/loader/cache/CachedSVGDocumentReference.h	Adds LoaderMalloc.h include for WTF_MAKE_FAST_ALLOCATED_WITH_HEAP_IDENTIFIER usage
Source/WebCore/animation/FrameRateAligner.cpp	Removes unnecessary DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER since it's not needed (class uses heap identifier through header)
Source/WTF/wtf/text/StringImpl.cpp	Adds NeverDestroyed.h include required by DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER macro

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.