[webkit-reviews] review denied: [Bug 230881] Have SourceBufferPrivateAVJObjC use WorkQueue instead of dispatch_async : [Attachment 440171] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Oct 5 07:38:07 PDT 2021

Chris Dumez <cdumez at apple.com> has denied Jean-Yves Avenard [:jya]
<jean-yves.avenard at apple.com>'s request for review:
Bug 230881: Have SourceBufferPrivateAVJObjC use WorkQueue instead of

Attachment 440171: Patch


--- Comment #7 from Chris Dumez <cdumez at apple.com> ---
Comment on attachment 440171
  --> https://bugs.webkit.org/attachment.cgi?id=440171

View in context: https://bugs.webkit.org/attachment.cgi?id=440171&action=review

> Source/WTF/wtf/cocoa/WorkQueueCocoa.cpp:59
> +    dispatch_group_enter(m_dispatchGroup.get());

I don't think this is the right level of abstraction and I don't think we want
to pay the cost of dispatch_group_enter / dispatch_group_leave for *every*
dispatch call, simply to support awaitIdle() at one call site.

If we really wanted to support this, I think we would introduce the concept of
group, like in the dispatch API. That said, for this patch, I don't think we
need the concept of group or awaitIdle() at all.

> +    m_appendQueue->awaitIdle();

Since you have a dedicated queue now, why can't you simply
m_appendQueue->dispatchSync() to make sure all previous tasks have completed
and hang while waiting?

The previous code was using the concept of group because we were using a shared
global queue.

More information about the webkit-reviews mailing list