compositor-worker: Initialize CW machinery plumbing to compositor and fire CW rAF callbacks.

third_party/WebKit/Source/web/CompositorProxyClientImpl.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
+
+#include "core/dom/CompositorProxyClient.h"
+

[jbroman, 2016/04/20 21:06:42]

nit: unnecessary newline

[flackr, 2016/04/25 14:06:28]

Removed.

+#include "wtf/Noncopyable.h"
+
+namespace blink {
+
+class CompositorMutatorImpl;
+class CompositorWorkerGlobalScope;
+class WorkerGlobalScope;
+
+class CompositorProxyClientImpl : public CompositorProxyClient {
+    WTF_MAKE_NONCOPYABLE(CompositorProxyClientImpl);
+
+public:
+    explicit CompositorProxyClientImpl(CompositorMutatorImpl*);
+    ~CompositorProxyClientImpl() override;
+    DECLARE_VIRTUAL_TRACE();
+
+    bool mutate(double timeNow) final;

[jbroman, 2016/04/20 21:06:42]

nit: this seems to be overriding a method in CompositorProxyClient. If it
continues to do so, please move it under the "CompositorProxyClient:" comment.

[flackr, 2016/04/25 14:06:28]

No longer overridiing the method in CompositorProxyClient.

+
+    // CompositorProxyClient:
+    void setGlobalScope(WorkerGlobalScope*) override;
+    void requestMutation() override;
+
+private:
+    bool executeAnimationFrameCallbacks(double time);
+
+    CompositorMutatorImpl* m_mutator;

[jbroman, 2016/04/20 21:06:42]

lifetime of the CompositorMutatorImpl isn't obvious; mind explaining why this
can't become a dangling pointer?

+
+    Member<CompositorWorkerGlobalScope> m_globalScope;
+    bool m_executingAnimationFrameCallbacks;
+    bool m_requestedAnimationFrameCallbacks;
+};
+
+} // namespace blink
+
+#endif // CompositorProxyClientImpl_h