[Nicosia] Add threaded PaintingEngine implementation
Created attachment 335421 [details] Patch
Comment on attachment 335421 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335421&action=review > Source/WebCore/platform/graphics/nicosia/NicosiaPaintingEngineThreaded.cpp:49 > + auto* taskData = static_cast<TaskData*>(data); We could adopt the data here using unique_ptr. > Source/WebCore/platform/graphics/nicosia/NicosiaPaintingEngineThreaded.cpp:97 > + paintLayer(context, layer, sourceRect, mappedSourceRect, targetRect, > + contentsScale, buffer->supportsAlpha()); I think this could be one line > Source/WebCore/platform/graphics/nicosia/NicosiaPaintingEngineThreaded.cpp:100 > + g_thread_pool_push(m_threadPool, new TaskData { WTFMove(buffer), WTFMove(paintingOperations) }, nullptr); I guess this happens a lot, can we make TaskData fast allocated?
Comment on attachment 335421 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335421&action=review >> Source/WebCore/platform/graphics/nicosia/NicosiaPaintingEngineThreaded.cpp:49 >> + auto* taskData = static_cast<TaskData*>(data); > > We could adopt the data here using unique_ptr. OK. >> Source/WebCore/platform/graphics/nicosia/NicosiaPaintingEngineThreaded.cpp:97 >> + contentsScale, buffer->supportsAlpha()); > > I think this could be one line OK. >> Source/WebCore/platform/graphics/nicosia/NicosiaPaintingEngineThreaded.cpp:100 >> + g_thread_pool_push(m_threadPool, new TaskData { WTFMove(buffer), WTFMove(paintingOperations) }, nullptr); > > I guess this happens a lot, can we make TaskData fast allocated? Makes sense.
Created attachment 335424 [details] Patch for landing
Comment on attachment 335424 [details] Patch for landing Clearing flags on attachment: 335424 Committed r229460: <https://trac.webkit.org/changeset/229460>
All reviewed patches have been landed. Closing bug.
<rdar://problem/38300730>