[ThreadedCompositor] Simply the compositing run loop worker thread
[WebKit-https.git] / Source / WebKit / ChangeLog
index 0e541ab..818c610 100644 (file)
@@ -1,3 +1,29 @@
+2019-03-01  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [ThreadedCompositor] Simply the compositing run loop worker thread
+        https://bugs.webkit.org/show_bug.cgi?id=195208
+
+        Reviewed by Don Olmstead.
+
+        We can remove the WorkQueuePool, since we never really supported more than one thread, and now that single
+        process model non longer exists it doesn't even make sense. We can simply use a RunLoop instead of a WorkQueue
+        so that the implementation is not specific to the generic WorkQueue implementation.
+
+        * Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.cpp:
+        (WebKit::createRunLoop): Helper function to create the RunLoop in a worker thread before m_updateTimer is initialized.
+        (WebKit::CompositingRunLoop::CompositingRunLoop): Use createRunLoop().
+        (WebKit::CompositingRunLoop::~CompositingRunLoop): Stop the worker thread run loop in the next main run loop iteration.
+        (WebKit::CompositingRunLoop::performTask): Use m_runLoop.
+        (WebKit::CompositingRunLoop::performTaskSync): Ditto.
+        (WebKit::WorkQueuePool::singleton): Deleted.
+        (WebKit::WorkQueuePool::dispatch): Deleted.
+        (WebKit::WorkQueuePool::runLoop): Deleted.
+        (WebKit::WorkQueuePool::invalidate): Deleted.
+        (WebKit::WorkQueuePool::WorkQueuePool): Deleted.
+        (WebKit::WorkQueuePool::getOrCreateWorkQueueForContext): Deleted.
+        (): Deleted.
+        * Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.h:
+
 2019-03-01  Justin Fan  <justin_fan@apple.com>
 
         [Web GPU] 32-bit builds broken by attempt to disable WebGPU on 32-bit