compositor-worker: Share a thread and an isolate for compositor workers.

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"
 
 namespace blink {
 
 class WorkerObjectProxy;
 
-class CompositorWorkerThread final : public WorkerThread {
+// This class is overridden in unit-tests.
+class CompositorWorkerThread : public WorkerThread {
 public:
     static PassRefPtr<CompositorWorkerThread> create(PassRefPtr<WorkerLoaderProxy>, WorkerObjectProxy&, double timeOrigin);
     virtual ~CompositorWorkerThread();
 
     WorkerObjectProxy& workerObjectProxy() const { return m_workerObjectProxy; }
 
-private:
+protected:
     CompositorWorkerThread(PassRefPtr<WorkerLoaderProxy>, WorkerObjectProxy&, double timeOrigin);
 
+private:
     // WorkerThread:
     PassRefPtrWillBeRawPtr<WorkerGlobalScope> createWorkerGlobalScope(PassOwnPtr<WorkerThreadStartupData>) override;
     WebThreadSupportingGC& backingThread() override;
+    PassOwnPtr<WorkerIsolateWrapper> createIsolateWrapper() override;
+    void didStartRunLoop() override { }
+    void didStopRunLoop() override { }
+    void initializeBackingThread() override;
+    void shutdownBackingThread() override;
+
+    WebThreadSupportingGC* m_thread = nullptr;

[haraken, 2015/06/02 02:14:49]

It looks nasty to keep a raw pointer to the WebThreadSupportingGC. Can we remove
this? (i.e., we can always use
CompositorWorkerManager::instance()->compositorWorkerThread() to get the Web
thread.)

[sadrul, 2015/06/02 02:48:28]

Done.

 
     WorkerObjectProxy& m_workerObjectProxy;
     double m_timeOrigin;
-    OwnPtr<WebThreadSupportingGC> m_thread;
 };
 
 } // namespace blink
 
 #endif // CompositorWorkerThread_h