[webkit-reviews] review granted: [Bug 226255] Stop using UncheckedLock in WTF::AutomaticThread : [Attachment 429727] Patch

[bugzilla-daemon at webkit.org]

Keith Miller <keith_miller at apple.com> has granted Chris Dumez
<cdumez at apple.com>'s request for review:
Bug 226255: Stop using UncheckedLock in WTF::AutomaticThread
https://bugs.webkit.org/show_bug.cgi?id=226255

Attachment 429727: Patch

https://bugs.webkit.org/attachment.cgi?id=429727&action=review




--- Comment #4 from Keith Miller <keith_miller at apple.com> ---
Comment on attachment 429727
  --> https://bugs.webkit.org/attachment.cgi?id=429727
Patch

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

> Source/WTF/wtf/ParallelHelperPool.cpp:189
> +	   assertIsHeld(*m_pool.m_lock);

Why not just annotate the poll function? Or does this only work for locks that
are a member of the same class as the method?