Add heap identifier macros to WebCore/animation

https://bugs.webkit.org/show_bug.cgi?id=260547 rdar://114281522 Reviewed by Brent Fulgham. Replace WTF_MAKE_FAST_ALLOCATED with WTF_MAKE_FAST_ALLOCATED_WITH_HEAP_IDENTIFIER for more fine-grained diagnostics when building with MallocHeapBreakdown enabled. * Source/WebCore/animation/AcceleratedTimeline.cpp: * Source/WebCore/animation/AcceleratedTimeline.h: * Source/WebCore/animation/CSSPropertyAnimation.cpp: * Source/WebCore/animation/DocumentTimelinesController.cpp: * Source/WebCore/animation/DocumentTimelinesController.h: * Source/WebCore/animation/ElementAnimationRareData.cpp: * Source/WebCore/animation/ElementAnimationRareData.h: * Source/WebCore/animation/FrameRateAligner.cpp: * Source/WebCore/animation/FrameRateAligner.h: * Source/WebCore/animation/KeyframeEffectStack.cpp: * Source/WebCore/animation/KeyframeEffectStack.h: Canonical link: https://commits.webkit.org/267159@main
WebKit · Aug 22, 2023 · c923b30 · c923b30
1 parent 0317cc8
commit c923b30
Show file tree

Hide file tree

Showing 11 changed files with 186 additions and 61 deletions.
diff --git a/Source/WebCore/animation/AcceleratedTimeline.cpp b/Source/WebCore/animation/AcceleratedTimeline.cpp
@@ -42,6 +42,7 @@
 #include <wtf/MonotonicTime.h>
 
 namespace WebCore {
+DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER(AcceleratedTimeline);
 
 AcceleratedTimeline::AcceleratedTimeline(Document& document)
 {

diff --git a/Source/WebCore/animation/AcceleratedTimeline.h b/Source/WebCore/animation/AcceleratedTimeline.h
@@ -37,8 +37,9 @@ class Document;
 class Element;
 struct Styleable;
 
+DECLARE_ALLOCATOR_WITH_HEAP_IDENTIFIER(AcceleratedTimeline);
 class AcceleratedTimeline {
-    WTF_MAKE_FAST_ALLOCATED;
+    WTF_MAKE_FAST_ALLOCATED_WITH_HEAP_IDENTIFIER(AcceleratedTimeline);
 public:
     AcceleratedTimeline(Document&);
 

diff --git a/Source/WebCore/animation/CSSPropertyAnimation.cpp b/Source/WebCore/animation/CSSPropertyAnimation.cpp
diff --git a/Source/WebCore/animation/DocumentTimelinesController.cpp b/Source/WebCore/animation/DocumentTimelinesController.cpp
@@ -41,6 +41,7 @@
 #include <wtf/text/TextStream.h>
 
 namespace WebCore {
+DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER(DocumentTimelinesController);
 
 DocumentTimelinesController::DocumentTimelinesController(Document& document)
     : m_document(document)

diff --git a/Source/WebCore/animation/DocumentTimelinesController.h b/Source/WebCore/animation/DocumentTimelinesController.h
@@ -40,8 +40,9 @@ class Document;
 class DocumentTimeline;
 class WebAnimation;
 
+DECLARE_ALLOCATOR_WITH_HEAP_IDENTIFIER(DocumentTimelinesController);
 class DocumentTimelinesController {
-    WTF_MAKE_FAST_ALLOCATED;
+    WTF_MAKE_FAST_ALLOCATED_WITH_HEAP_IDENTIFIER(DocumentTimelinesController);
 public:
     explicit DocumentTimelinesController(Document&);
     ~DocumentTimelinesController();

diff --git a/Source/WebCore/animation/ElementAnimationRareData.cpp b/Source/WebCore/animation/ElementAnimationRareData.cpp
@@ -32,6 +32,7 @@
 #include "RenderStyle.h"
 
 namespace WebCore {
+DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER(ElementAnimationRareData);
 
 ElementAnimationRareData::ElementAnimationRareData(PseudoId pseudoId)
     : m_pseudoId(pseudoId)

diff --git a/Source/WebCore/animation/ElementAnimationRareData.h b/Source/WebCore/animation/ElementAnimationRareData.h
@@ -35,9 +35,10 @@ class CSSTransition;
 class RenderStyle;
 class WebAnimation;
 
+DECLARE_ALLOCATOR_WITH_HEAP_IDENTIFIER(ElementAnimationRareData);
 class ElementAnimationRareData {
+    WTF_MAKE_FAST_ALLOCATED_WITH_HEAP_IDENTIFIER(ElementAnimationRareData);
     WTF_MAKE_NONCOPYABLE(ElementAnimationRareData);
-    WTF_MAKE_FAST_ALLOCATED;
 public:
     explicit ElementAnimationRareData(PseudoId);
     ~ElementAnimationRareData();

diff --git a/Source/WebCore/animation/FrameRateAligner.cpp b/Source/WebCore/animation/FrameRateAligner.cpp
@@ -27,6 +27,7 @@
 #include "FrameRateAligner.h"
 
 namespace WebCore {
+DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER(FrameRateAligner);
 
 FrameRateAligner::FrameRateAligner() = default;
 

diff --git a/Source/WebCore/animation/FrameRateAligner.h b/Source/WebCore/animation/FrameRateAligner.h
@@ -32,8 +32,9 @@
 
 namespace WebCore {
 
+DECLARE_ALLOCATOR_WITH_HEAP_IDENTIFIER(FrameRateAligner);
 class FrameRateAligner {
-    WTF_MAKE_FAST_ALLOCATED;
+    WTF_MAKE_FAST_ALLOCATED_WITH_HEAP_IDENTIFIER(FrameRateAligner);
 public:
     FrameRateAligner();
     ~FrameRateAligner();

diff --git a/Source/WebCore/animation/KeyframeEffectStack.cpp b/Source/WebCore/animation/KeyframeEffectStack.cpp
@@ -42,6 +42,7 @@
 #include <wtf/PointerComparison.h>
 
 namespace WebCore {
+DEFINE_ALLOCATOR_WITH_HEAP_IDENTIFIER(KeyframeEffectStack);
 
 KeyframeEffectStack::KeyframeEffectStack()
 {

diff --git a/Source/WebCore/animation/KeyframeEffectStack.h b/Source/WebCore/animation/KeyframeEffectStack.h
@@ -41,8 +41,9 @@ namespace Style {
 struct ResolutionContext;
 }
 
+DECLARE_ALLOCATOR_WITH_HEAP_IDENTIFIER(KeyframeEffectStack);
 class KeyframeEffectStack {
-    WTF_MAKE_FAST_ALLOCATED;
+    WTF_MAKE_FAST_ALLOCATED_WITH_HEAP_IDENTIFIER(KeyframeEffectStack);
 public:
     explicit KeyframeEffectStack();
     ~KeyframeEffectStack();