Bug 171146

Summary: [MediaCapture] Improvements to CoreAudioCaptureSource
Product: WebKit Reporter: Jer Noble <jer.noble>
Component: New BugsAssignee: Jer Noble <jer.noble>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric.carlson, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch for Landing none

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.