Use wrapper tracing for worklets.

third_party/WebKit/Source/modules/compositorworker/AnimationWorkletGlobalScope.h

Index: third_party/WebKit/Source/modules/compositorworker/AnimationWorkletGlobalScope.h
diff --git a/third_party/WebKit/Source/modules/compositorworker/AnimationWorkletGlobalScope.h b/third_party/WebKit/Source/modules/compositorworker/AnimationWorkletGlobalScope.h
index e9fabbd5886d4468209fc928f17e43109e612810..c4c4eccd8658f3ef384a59e0788f340d653c5054 100644
--- a/third_party/WebKit/Source/modules/compositorworker/AnimationWorkletGlobalScope.h
+++ b/third_party/WebKit/Source/modules/compositorworker/AnimationWorkletGlobalScope.h
@@ -9,6 +9,7 @@
 #include "core/workers/ThreadedWorkletGlobalScope.h"
 #include "modules/compositorworker/Animator.h"
 #include "modules/compositorworker/AnimatorDefinition.h"
+#include "platform/bindings/ScriptWrappable.h"
 
 namespace blink {
 
@@ -25,8 +26,7 @@ class AnimationWorkletGlobalScope : public ThreadedWorkletGlobalScope {
                                              WorkerThread*);
   ~AnimationWorkletGlobalScope() override;
   DECLARE_TRACE();
-
-  void Dispose() final;
+  DECLARE_TRACE_WRAPPERS();
 
   void registerAnimator(const String& name,
                         const ScriptValue& ctorValue,
@@ -41,11 +41,12 @@ class AnimationWorkletGlobalScope : public ThreadedWorkletGlobalScope {
                               v8::Isolate*,
                               WorkerThread*);
 
-  typedef HeapHashMap<String, Member<AnimatorDefinition>> DefinitionMap;
-  DefinitionMap m_animatorDefinitions;
+  typedef HeapHashMap<String, TraceWrapperMember<AnimatorDefinition>>
+      DefinitionMap;
+  DefinitionMap animator_definitions_;
 
-  typedef HeapVector<Member<Animator>> AnimatorList;
-  AnimatorList m_animators;
+  typedef HeapVector<TraceWrapperMember<Animator>> AnimatorList;
+  AnimatorList animators_;

[haraken, 2017/05/25 04:22:12]

Do you need to trace both animator_definitions_ and animators_? It's harmless to
trace both but it's sufficient to trace AnimatorDefinition once. Maybe would it
be sufficient to trace only animators_? Then the Animator object will trace into
the corresponding AnimatorDefinition.

[majidvp, 2017/05/25 20:16:57]

hmmmm, I think it is better to trace them both here. 

If you want to avoid overhead of tracing things twice (*) then I propose to not
trace the definition from animator. My reasoning is that both animators and
definitions are owned by the scope. The animator reference to its definition is
not owning one. So it seems more natural to trace along the ownership graph.

(*) I am not sure why we care about the tracing cost. In particular, for oilpan
tracing it is very typical to have the
same object being traced via Member<> from multiple path and that
is not very concerning. Is there anything special about wrapper
tracing to warrant being careful as opposed to a simpler guideline
of trace every wrapper.

 };
 
 }  // namespace blink