Prepare CompositorWorker for per thread heap

third_party/WebKit/Source/web/CompositorProxyClientImpl.h

Index: third_party/WebKit/Source/web/CompositorProxyClientImpl.h
diff --git a/third_party/WebKit/Source/web/CompositorProxyClientImpl.h b/third_party/WebKit/Source/web/CompositorProxyClientImpl.h
index 3390c4a965356cfb6abb1ee876835b7fa999ae44..e3fb0582b092ea228d11107563f5b26be5e9ebd3 100644
--- a/third_party/WebKit/Source/web/CompositorProxyClientImpl.h
+++ b/third_party/WebKit/Source/web/CompositorProxyClientImpl.h
@@ -21,7 +21,7 @@ class WorkerGlobalScope;
 // mutator, e.g. if a single document creates multiple CompositorWorker objects.
 //
 // Should be accessed only on the compositor thread.
-class CompositorProxyClientImpl final : public GarbageCollected<CompositorProxyClientImpl>, public CompositorProxyClient {
+class CompositorProxyClientImpl final : public GarbageCollectedFinalized<CompositorProxyClientImpl>, public CompositorProxyClient {
     USING_GARBAGE_COLLECTED_MIXIN(CompositorProxyClientImpl);
     WTF_MAKE_NONCOPYABLE(CompositorProxyClientImpl);
 public:
@@ -41,9 +41,9 @@ public:
 private:
     bool executeAnimationFrameCallbacks(double monotonicTimeNow);
 
-    Member<CompositorMutatorImpl> m_mutator;
+    CrossThreadPersistent<CompositorMutatorImpl> m_mutator;
 
-    Member<CompositorWorkerGlobalScope> m_globalScope;
+    CrossThreadWeakPersistent<CompositorWorkerGlobalScope> m_globalScope;

[keishi, 2016/09/02 06:41:55]

CompositorProxyClientImpl::m_globalScope was causing a leak because
CompositorProxyClientImpl supplements WorkerClients which is referenced from
WorkerGlobalScope. WorkerGlobalScope should outlive this object so I think its
safe to make weak.

     bool m_requestedAnimationFrameCallbacks;
 
     HeapHashSet<WeakMember<CompositorProxy>> m_proxies;