CompositorWorkerProxyClientImpl to CompositorProxy cross thread reference is using WeakMember

third_party/WebKit/Source/core/dom/AnimationWorkletProxyClient.h

Index: third_party/WebKit/Source/core/dom/AnimationWorkletProxyClient.h
diff --git a/third_party/WebKit/Source/core/dom/AnimationWorkletProxyClient.h b/third_party/WebKit/Source/core/dom/AnimationWorkletProxyClient.h
index 2a3a9051bab0ee6e4753a856b9a209fba9762095..d06108496a7d2344e2df06d337538cec1d5582e6 100644
--- a/third_party/WebKit/Source/core/dom/AnimationWorkletProxyClient.h
+++ b/third_party/WebKit/Source/core/dom/AnimationWorkletProxyClient.h
@@ -11,11 +11,14 @@
 
 namespace blink {
 
-class CORE_EXPORT AnimationWorkletProxyClient : public CompositorProxyClient {
+class CORE_EXPORT AnimationWorkletProxyClient

[sof, 2017/03/29 05:51:05]

What role does this abstraction play? i.e., it's now a synonym for GCFinalized.

[keishi, 2017/03/30 08:34:14]

I think it's suppose to be an interface so it can be accessed from core/ or
modules/. AnimationWorkletProxyClientImpl is in web/ and
AnimationWorkletProxyClient is used in modules/. I don't think I can delete this
class without upsetting DEPS.

Changed to GarbageCollectedMixin.

+    : public GarbageCollectedFinalized<AnimationWorkletProxyClient> {
   WTF_MAKE_NONCOPYABLE(AnimationWorkletProxyClient);
 
  public:
   AnimationWorkletProxyClient() {}
+  virtual ~AnimationWorkletProxyClient() {}
+  DEFINE_INLINE_VIRTUAL_TRACE() {}
 };
 
 }  // namespace blink