[MediaCapture] Improvements to CoreAudioCaptureSource
Created attachment 307819 [details] Patch
Created attachment 307820 [details] Patch
Created attachment 307826 [details] Patch
Created attachment 307827 [details] Patch
Created attachment 307837 [details] Patch
Comment on attachment 307837 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=307837&action=review > Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:402 > + uint32_t propertySize = sizeof(m_captureDeviceID); Does this work in a 32-bit build? > Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:550 > + m_currentSettings = std::nullopt; This should call the base class method so it notifies observers.
<rdar://problem/31769428>
(In reply to Eric Carlson from comment #6) > Comment on attachment 307837 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=307837&action=review > > > Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:402 > > + uint32_t propertySize = sizeof(m_captureDeviceID); > > Does this work in a 32-bit build? It should, since we use this pattern in other places in this file. > > Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:550 > > + m_currentSettings = std::nullopt; > > This should call the base class method so it notifies observers. Good catch!
Created attachment 307845 [details] Patch for Landing
Committed r215659: <http://trac.webkit.org/changeset/215659>