RESOLVED FIXED 171146
[MediaCapture] Improvements to CoreAudioCaptureSource
https://bugs.webkit.org/show_bug.cgi?id=171146
Summary [MediaCapture] Improvements to CoreAudioCaptureSource
Jer Noble
Reported 2017-04-21 15:22:50 PDT
[MediaCapture] Improvements to CoreAudioCaptureSource
Attachments
Patch (13.65 KB, patch)
2017-04-21 15:30 PDT, Jer Noble
no flags
Patch (11.02 KB, patch)
2017-04-21 15:38 PDT, Jer Noble
no flags
Patch (11.02 KB, patch)
2017-04-21 16:18 PDT, Jer Noble
no flags
Patch (11.02 KB, patch)
2017-04-21 16:20 PDT, Jer Noble
no flags
Patch (13.21 KB, patch)
2017-04-21 16:41 PDT, Jer Noble
no flags
Patch for Landing (13.25 KB, patch)
2017-04-21 17:25 PDT, Jer Noble
no flags
Jer Noble
Comment 1 2017-04-21 15:30:50 PDT
Jer Noble
Comment 2 2017-04-21 15:38:07 PDT
Jer Noble
Comment 3 2017-04-21 16:18:57 PDT
Jer Noble
Comment 4 2017-04-21 16:20:54 PDT
Jer Noble
Comment 5 2017-04-21 16:41:27 PDT
Eric Carlson
Comment 6 2017-04-21 16:54:48 PDT
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.
Radar WebKit Bug Importer
Comment 7 2017-04-21 16:55:45 PDT
Jer Noble
Comment 8 2017-04-21 17:22:19 PDT
(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!
Jer Noble
Comment 9 2017-04-21 17:25:54 PDT
Created attachment 307845 [details] Patch for Landing
Jer Noble
Comment 10 2017-04-21 18:00:20 PDT
Note You need to log in before you can comment on or make changes to this bug.