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

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

 // 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.
 
 #include "modules/compositorworker/CompositorWorker.h"
 
 #include "bindings/core/v8/ExceptionState.h"
+#include "core/dom/CompositorProxyClient.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
+#include "core/page/ChromeClient.h"
 #include "core/workers/WorkerClients.h"
 #include "modules/EventTargetModules.h"
 #include "modules/compositorworker/CompositorWorkerMessagingProxy.h"
+#include "modules/compositorworker/CompositorWorkerThread.h"
 
 namespace blink {
 
 inline CompositorWorker::CompositorWorker(ExecutionContext* context)
     : InProcessWorkerBase(context)
 {
 }
 
 CompositorWorker* CompositorWorker::create(ExecutionContext* context, const String& url, ExceptionState& exceptionState)
 {
+    // Ensure the compositor worker backing thread is ready.
+    CompositorWorkerThread::ensureSharedBackingThread();
+
     ASSERT(isMainThread());
     Document* document = toDocument(context);
     if (!document->page()) {
         exceptionState.throwDOMException(InvalidAccessError, "The context provided is invalid.");
         return nullptr;
     }
     CompositorWorker* worker = new CompositorWorker(context);
     if (worker->initialize(context, url, exceptionState))
         return worker;
     return nullptr;
 }
 
 CompositorWorker::~CompositorWorker()
 {
     ASSERT(isMainThread());
 }
 
 const AtomicString& CompositorWorker::interfaceName() const
 {
     return EventTargetNames::CompositorWorker;
 }
 
-WorkerGlobalScopeProxy* CompositorWorker::createWorkerGlobalScopeProxy(ExecutionContext* worker)
+WorkerGlobalScopeProxy* CompositorWorker::createWorkerGlobalScopeProxy(ExecutionContext* context)
 {
-    ASSERT(getExecutionContext()->isDocument());
-    return new CompositorWorkerMessagingProxy(this);
+    ASSERT(context->isDocument());

[jbroman, 2016/04/29 19:42:22]

The assertion here is redundant, since toDocument does the same assertion
internally. The fact that you call toDocument already documents -- if you'll
excuse the pun -- the underlying type you expect for the ExecutionContext.

[flackr, 2016/05/03 22:24:54]

Heh, I like the pun. Thanks, removed.

+    Document* document = toDocument(context);
+    WorkerClients* workerClients = WorkerClients::create();
+    provideCompositorProxyClientTo(workerClients, document->frame()->chromeClient().createCompositorProxyClient(document->frame()));
+    return new CompositorWorkerMessagingProxy(this, workerClients);
 }
 
 } // namespace blink