Bug 172512

Summary:

CoreAudioCaptureSource is ducking system sound

Product:

WebKit

Reporter:

youenn fablet <youennf>

Component:

Media

Assignee:

youenn fablet <youennf>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

buildbot, commit-queue, eric.carlson, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

172538

Bug Blocks:

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Adding CoreAudio as framework for ios-sim as well	none
Patch	none
Patch	none
Patch	none

youenn fablet

Reported 2017-05-23 11:11:29 PDT

This happens on some Macs

Attachments
Patch (1.54 KB, patch) 2017-05-23 11:13 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (2.69 KB, patch) 2017-05-23 16:12 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (19.03 KB, patch) 2017-05-23 16:34 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (2.95 KB, patch) 2017-05-23 16:54 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (3.00 KB, patch) 2017-05-23 17:02 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (3.50 KB, patch) 2017-05-23 17:28 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (3.21 KB, patch) 2017-05-23 20:00 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (3.28 KB, patch) 2017-05-23 20:44 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (3.53 KB, patch) 2017-05-23 20:55 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (3.54 KB, patch) 2017-05-23 21:12 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (3.34 KB, patch) 2017-05-23 21:34 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Adding CoreAudio as framework for ios-sim as well (6.09 KB, patch) 2017-05-24 08:32 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (11.01 KB, patch) 2017-05-24 10:30 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (11.21 KB, patch) 2017-05-24 10:58 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Patch (11.25 KB, patch) 2017-05-24 11:11 PDT, youenn fablet	no flags	Details Formatted Diff Diff
Show Obsolete (14) View All Add attachment proposed patch, testcase, etc.

youenn fablet

Comment 1 2017-05-23 11:13:23 PDT

Created attachment 311029 [details] Patch

youenn fablet

Comment 2 2017-05-23 16:12:06 PDT

Created attachment 311071 [details] Patch

Build Bot

Comment 3 2017-05-23 16:14:05 PDT

Attachment 311071 [details] did not pass style-queue: ERROR: Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:50: *SoftLink.h header should be included after all other headers. [build/include_order] [4] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.

youenn fablet

Comment 4 2017-05-23 16:34:14 PDT

Created attachment 311073 [details] Patch

Build Bot

Comment 5 2017-05-23 16:40:08 PDT

Attachment 311073 [details] did not pass style-queue: ERROR: Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:53: *SoftLink.h header should be included after all other headers. [build/include_order] [4] Total errors found: 1 in 17 files If any of these errors are false positives, please file a bug against check-webkit-style.

youenn fablet

Comment 6 2017-05-23 16:54:01 PDT

Created attachment 311076 [details] Patch

Build Bot

Comment 7 2017-05-23 16:55:10 PDT

Attachment 311076 [details] did not pass style-queue: ERROR: Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:53: *SoftLink.h header should be included after all other headers. [build/include_order] [4] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.

youenn fablet

Comment 8 2017-05-23 17:02:55 PDT

Created attachment 311079 [details] Patch

Build Bot

Comment 9 2017-05-23 17:05:34 PDT

Attachment 311079 [details] did not pass style-queue: ERROR: Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:53: *SoftLink.h header should be included after all other headers. [build/include_order] [4] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.

youenn fablet

Comment 10 2017-05-23 17:28:06 PDT

Created attachment 311084 [details] Patch

Build Bot

Comment 11 2017-05-23 17:29:57 PDT

Attachment 311084 [details] did not pass style-queue: ERROR: Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.cpp:54: *SoftLink.h header should be included after all other headers. [build/include_order] [4] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.

youenn fablet

Comment 12 2017-05-23 20:00:32 PDT

Created attachment 311092 [details] Patch

youenn fablet

Comment 13 2017-05-23 20:44:37 PDT

Created attachment 311095 [details] Patch

youenn fablet

Comment 14 2017-05-23 20:55:06 PDT

Created attachment 311096 [details] Patch

youenn fablet

Comment 15 2017-05-23 21:12:42 PDT

Created attachment 311097 [details] Patch