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

third_party/WebKit/Source/modules/compositorworker/CompositorWorkerThread.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 CompositorWorkerThread_h
 #define CompositorWorkerThread_h
 
 #include "core/workers/WorkerThread.h"
 #include "modules/ModulesExport.h"
 
 namespace blink {
 
 class InProcessWorkerObjectProxy;
 
 // This class is overridden in unit-tests.
 class MODULES_EXPORT CompositorWorkerThread final : public WorkerThread {
 public:
     static PassOwnPtr<CompositorWorkerThread> create(PassRefPtr<WorkerLoaderProxy>, InProcessWorkerObjectProxy&, double timeOrigin);
     ~CompositorWorkerThread() override;
 
     InProcessWorkerObjectProxy& workerObjectProxy() const { return m_workerObjectProxy; }
     WorkerBackingThread& workerBackingThread() override;
     bool shouldAttachThreadDebugger() const override { return false; }
 
+    static void ensureSharedBackingThread();
     static void resetSharedBackingThreadForTest();
+    static void clearSharedBackingThread();

[haraken, 2016/05/06 02:45:51]

+yhirano since he removed this method last week.

[yhirano, 2016/05/06 05:47:43]

Please add "ForTest" as this function is used only in tests.

Can you tell me why you need this function in CompositorWorkerThreadTest?

[flackr, 2016/05/06 18:15:07]

Ah, I didn't notice that you had removed the call to it from ModulesInitializer
when I merged it in. I was expecting to shut down the compositor worker thread
in ModulesInitializer::shutdown if it had been instantiated. I've now added this
to the patch.
Using CompositorWorkerThread::resetSharedBackingThreadForTest() causes a new
WorkerBackingThread to be created for the current "compositor thread". In
CompositorWorkerThreadTest, this thread is destroyed between tests so when we
call resetSharedBackingThreadForTest we try to post a task to this thread which
no longer exists.

[yhirano, 2016/05/09 03:15:31]

See https://codereview.chromium.org/1733353004/#msg77. It's more problematic
with this CL because at that time the code only allocated and destructed
WebThreadSupportingGC but now it runs the shutdown sequence.
Then I think you need to clear out BackingThreadHolder's state more deeply
because after calling clearSharedBackingThread() it's no longer usable. Calling
clearSharedBackingThread() means subsequent tests need to call
resetSharedBackingThreadForTest before using it and that is very confusing.

 
 protected:
     CompositorWorkerThread(PassRefPtr<WorkerLoaderProxy>, InProcessWorkerObjectProxy&, double timeOrigin);
 
     WorkerGlobalScope* createWorkerGlobalScope(PassOwnPtr<WorkerThreadStartupData>) override;
 
 private:
     InProcessWorkerObjectProxy& m_workerObjectProxy;
     double m_timeOrigin;
 };
 
 } // namespace blink
 
 #endif // CompositorWorkerThread_h