Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1547)

Unified Diff: Source/modules/compositorworker/CompositorWorkerManager.h

Issue 1158443008: compositor-worker: Share a thread and an isolate for compositor workers. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: . Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/modules/compositorworker/CompositorWorkerManager.h
diff --git a/Source/modules/compositorworker/CompositorWorkerManager.h b/Source/modules/compositorworker/CompositorWorkerManager.h
new file mode 100644
index 0000000000000000000000000000000000000000..69f1b5543edd6c1807a9123a97918ec04471e72c
--- /dev/null
+++ b/Source/modules/compositorworker/CompositorWorkerManager.h
@@ -0,0 +1,54 @@
+// 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 CompositorWorkerManager_h
+#define CompositorWorkerManager_h
+
+#include "core/workers/WorkerV8Isolate.h"
+#include "modules/ModulesExport.h"
+#include "wtf/OwnPtr.h"
+#include "wtf/PassOwnPtr.h"
+#include "wtf/ThreadingPrimitives.h"
+#include <v8.h>
+
+namespace blink {
+
+class WebThreadSupportingGC;
+
+class MODULES_EXPORT CompositorWorkerManager final {
+public:
+ static void initialize();
+ static void shutdown();
+
+ static CompositorWorkerManager* instance();
+
+ // Returns the thread used for compositor workers. This creates a new thread if a
+ // thread doesn't already exist.
+ WebThreadSupportingGC& compositorWorkerThread();
+
+ // Attempts to initialize/shutdown a thread if necessary. Does nothing if the thread
+ // is already initialized, or if the thread has more than one active workers at the
+ // time of shutdown.
+ void initializeBackingThread();
+ void shutdownBackingThread();
+
+ PassOwnPtr<WorkerV8Isolate> createIsolate();
+ void willDestroyIsolate();
+ void destroyIsolate();
+ void terminateIsolate();
+
+private:
+ CompositorWorkerManager();
+ ~CompositorWorkerManager();
+
+ Mutex m_mutex;
+ OwnPtr<WebThreadSupportingGC> m_thread;
+ int m_workerCount = 0;
+ OwnPtr<WorkerV8Isolate> m_isolate;
+ int m_isolateCount = 0;
kinuko 2015/05/31 15:18:24 Using manual counters is not great especially if w
sadrul 2015/06/01 05:24:11 Done.
+};
+
+} // namespace blink
+
+#endif // CompositorWorkerManager_h

Powered by Google App Engine
This is Rietveld 408576698