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

third_party/WebKit/Source/modules/compositorworker/AnimationWorklet.cpp

 // Copyright 2016 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.
 
 #include "modules/compositorworker/AnimationWorklet.h"
 
 #include "bindings/core/v8/V8Binding.h"
+#include "core/dom/AnimationWorkletProxyClient.h"
 #include "core/dom/Document.h"
 #include "core/frame/LocalFrame.h"
+#include "core/page/ChromeClient.h"
 #include "modules/compositorworker/AnimationWorkletMessagingProxy.h"
 #include "modules/compositorworker/AnimationWorkletThread.h"
 
 namespace blink {
 
 // static
 AnimationWorklet* AnimationWorklet::create(LocalFrame* frame) {
   return new AnimationWorklet(frame);
 }
 
 AnimationWorklet::AnimationWorklet(LocalFrame* frame)
     : Worklet(frame), m_workletMessagingProxy(nullptr) {}
 
 AnimationWorklet::~AnimationWorklet() {
   if (m_workletMessagingProxy)
     m_workletMessagingProxy->parentObjectDestroyed();
 }
 
 void AnimationWorklet::initialize() {
   AnimationWorkletThread::ensureSharedBackingThread();
 
   DCHECK(!m_workletMessagingProxy);
   DCHECK(getExecutionContext());
+  Document* document = toDocument(getExecutionContext());
+  CompositorProxyClient* client =
+      document->frame()->chromeClient().createCompositorProxyClient(
+          CompositorProxyClient::kWorkletClient, document->frame());
+  AnimationWorkletProxyClient* proxyClient =
+      static_cast<AnimationWorkletProxyClient*>(client);

[dcheng, 2017/01/24 22:56:26]

Rather than downcasting here and below, we should just have two separate
functions IMO. While it costs us an extra vtable entry, it's more robust against
programmer error.

[majidvp, 2017/01/26 15:00:04]

This means adding two methods to more than a few places (ChromeClient and its
impls RenderWidgetFrameBase and its impls). Perhaps we can achieve the same
robustness by doing safer typecasts via DEFINE_TYPE_CASTS? This seems to be a
fairly typical pattern in Blink. I prefer this option if you are open to it.

 
   m_workletMessagingProxy =
-      new AnimationWorkletMessagingProxy(getExecutionContext());
+      new AnimationWorkletMessagingProxy(getExecutionContext(), proxyClient);
   m_workletMessagingProxy->initialize();
 }
 
 bool AnimationWorklet::isInitialized() const {
   return m_workletMessagingProxy;
 }
 
 WorkletGlobalScopeProxy* AnimationWorklet::workletGlobalScopeProxy() const {
   DCHECK(m_workletMessagingProxy);
   return m_workletMessagingProxy;
 }
 
 DEFINE_TRACE(AnimationWorklet) {
   Worklet::trace(visitor);
 }
 
 }  // namespace blink