There is no need to ref MediaStreamTrackPrivate on a background thread
Created attachment 377013 [details] Patch
Comment on attachment 377013 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=377013&action=review I’m going to say review+, but I am just taking your word for it that this is correct. > Source/WebCore/ChangeLog:3 > + Make MediaStreamTrackPrivate RefCounted Really need a "why" comment in the change log. I think it’s "MediaStreamTrackPrivate does not need to be ThreadSafeRefCounted because all use of these objects including ref/deref is on the main thread."
Created attachment 377240 [details] Improved change log
Comment on attachment 377240 [details] Improved change log Clearing flags on attachment: 377240 Committed r249134: <https://trac.webkit.org/changeset/249134>
All reviewed patches have been landed. Closing bug.
<rdar://problem/54738510>