Introduce AnimationWorkletProxyClient and necessary plumbing to get it in worklet messaging proxy.

third_party/WebKit/Source/web/CompositorWorkerProxyClientImpl.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CompositorProxyClientImpl_h
-#define CompositorProxyClientImpl_h
+#ifndef CompositorWorkerProxyClientImpl_h
+#define CompositorWorkerProxyClientImpl_h
 
-#include "core/dom/CompositorProxyClient.h"
+#include "core/dom/CompositorWorkerProxyClient.h"
+#include "platform/graphics/CompositorAnimator.h"
 #include "platform/heap/Handle.h"
 #include "wtf/Noncopyable.h"
 
 namespace blink {
 
 class CompositorMutableStateProvider;
 class CompositorMutatorImpl;
 class CompositorWorkerGlobalScope;
 class WorkerGlobalScope;
 
 // Mediates between one CompositorWorkerGlobalScope and the associated
-// CompositorMutatorImpl.  There is one CompositorProxyClientImpl per worker
-// but there may be multiple for a given mutator, e.g. if a single document
-// creates multiple CompositorWorker objects.
+// CompositorMutatorImpl. There is one CompositorWorkerProxyClientImpl per
+// worker but there may be multiple for a given mutator, e.g. if a single
+// document creates multiple CompositorWorker objects.
 //
 // Should be accessed only on the compositor thread.
-class CompositorProxyClientImpl final
-    : public GarbageCollectedFinalized<CompositorProxyClientImpl>,
-      public CompositorProxyClient {
-  USING_GARBAGE_COLLECTED_MIXIN(CompositorProxyClientImpl);
-  WTF_MAKE_NONCOPYABLE(CompositorProxyClientImpl);
+class CompositorWorkerProxyClientImpl final
+    : public GarbageCollectedFinalized<CompositorWorkerProxyClientImpl>,
+      public CompositorAnimator,
+      public CompositorWorkerProxyClient {
+  WTF_MAKE_NONCOPYABLE(CompositorWorkerProxyClientImpl);
+  USING_GARBAGE_COLLECTED_MIXIN(CompositorWorkerProxyClientImpl);
 
  public:
-  CompositorProxyClientImpl(CompositorMutatorImpl*);
-  DECLARE_VIRTUAL_TRACE();
+  CompositorWorkerProxyClientImpl(CompositorMutatorImpl*);

[ikilpatrick, 2016/11/28 16:43:42]

explicit?

[majidvp, 2017/01/20 15:48:57]

Done.

+  DECLARE_TRACE();
 
-  // Runs the animation frame callback for the frame starting at the given time.
-  // Returns true if another animation frame was requested (i.e. should be
-  // reinvoked next frame).
-  bool mutate(double monotonicTimeNow, CompositorMutableStateProvider*);
+  // CompositorAnimator:
+  bool mutate(double monotonicTimeNow,
+              CompositorMutableStateProvider*) override;
 
-  // CompositorProxyClient:
+  // CompositorWorkerProxyClient:
   void dispose() override;
   void setGlobalScope(WorkerGlobalScope*) override;
   void requestAnimationFrame() override;
   void registerCompositorProxy(CompositorProxy*) override;
   void unregisterCompositorProxy(CompositorProxy*) override;
 
  private:
   bool executeAnimationFrameCallbacks(double monotonicTimeNow);
 
   CrossThreadPersistent<CompositorMutatorImpl> m_mutator;
 
   CrossThreadPersistent<CompositorWorkerGlobalScope> m_globalScope;
   bool m_requestedAnimationFrameCallbacks;
 
+  // TODO(majidvp): move this out to a separate class that can be composed in.
   HeapHashSet<WeakMember<CompositorProxy>> m_proxies;
 };
 
 }  // namespace blink
 
-#endif  // CompositorProxyClientImpl_h
+#endif  // CompositorWorkerProxyClientImpl_h