Hop to main thread for release logging in RealtimeIncomingAudioSource
<rdar://problem/53127693>
Created attachment 374163 [details] patch
Comment on attachment 374163 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=374163&action=review > Source/WebCore/ChangeLog:3 > + Hop to main thread for release logging in RealtimeIncomingAudioSource Could you please explain why in the change loge? Isn't it safe to call RELEASE_LOG() from another thread?
Created attachment 374166 [details] patch
(In reply to Chris Dumez from comment #3) > Comment on attachment 374163 [details] > patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=374163&action=review > > > Source/WebCore/ChangeLog:3 > > + Hop to main thread for release logging in RealtimeIncomingAudioSource > > Could you please explain why in the change loge? Isn't it safe to call > RELEASE_LOG() from another thread? It is not thread safe, see Logger::log in Logger.h. I updated the change log.
Comment on attachment 374166 [details] patch Clearing flags on attachment: 374166 Committed r247480: <https://trac.webkit.org/changeset/247480>
All reviewed patches have been landed. Closing bug.