This is useless otherwise
Created attachment 354395 [details] Patch
Created attachment 354413 [details] Patch
Created attachment 354434 [details] Patch
Created attachment 354438 [details] Patch
Comment on attachment 354438 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=354438&action=review > Source/WebCore/platform/mediastream/RealtimeOutgoingAudioSource.cpp:65 > + observeSource(); As we discussed, this shouldn't be necessary because you also call it in AddSink. > Source/WebCore/platform/mediastream/RealtimeOutgoingVideoSource.cpp:73 > + observeSource(); Ditto. > Source/WebCore/platform/mediastream/RealtimeOutgoingVideoSource.cpp:81 > m_videoSource->removeObserver(*this); Nit: why does RealtimeOutgoingAudioSource have unobserveSource and RealtimeOutgoingVideoSource just calls removeObserver?
Thanks for the review. (In reply to Eric Carlson from comment #5) > Comment on attachment 354438 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=354438&action=review > > > Source/WebCore/platform/mediastream/RealtimeOutgoingAudioSource.cpp:65 > > + observeSource(); > > As we discussed, this shouldn't be necessary because you also call it in > AddSink. Yes, I will lock the sinks so that we observe/unobserve appropriately. > > Source/WebCore/platform/mediastream/RealtimeOutgoingVideoSource.cpp:73 > > + observeSource(); > > Ditto. > > > Source/WebCore/platform/mediastream/RealtimeOutgoingVideoSource.cpp:81 > > m_videoSource->removeObserver(*this); > > Nit: why does RealtimeOutgoingAudioSource have unobserveSource and > RealtimeOutgoingVideoSource just calls removeObserver? I will add unobserveSource.
Created attachment 354571 [details] Patch for landing
Comment on attachment 354571 [details] Patch for landing Clearing flags on attachment: 354571 Committed r238102: <https://trac.webkit.org/changeset/238102>
All reviewed patches have been landed. Closing bug.
<rdar://problem/45998054>