Bug 198307 - Add an option to mute audio capture automatically when page is not visible
Summary: Add an option to mute audio capture automatically when page is not visible
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebRTC (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-05-28 15:18 PDT by youenn fablet
Modified: 2019-05-31 16:49 PDT (History)
7 users (show)

See Also:


Attachments
Patch (33.07 KB, patch)
2019-05-28 16:57 PDT, youenn fablet
no flags Details | Formatted Diff | Diff
Patch (32.36 KB, patch)
2019-05-29 08:15 PDT, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (32.32 KB, patch)
2019-05-29 10:34 PDT, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (33.64 KB, patch)
2019-05-30 12:58 PDT, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (33.67 KB, patch)
2019-05-30 15:46 PDT, youenn fablet
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews125 for ios-simulator-wk2 (3.14 MB, application/zip)
2019-05-30 18:49 PDT, EWS Watchlist
no flags Details
Fixing watchOS build (33.68 KB, patch)
2019-05-31 09:08 PDT, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2019-05-28 15:18:49 PDT
Add an option to mute audio capture automatically when page is not visible
Comment 1 youenn fablet 2019-05-28 16:57:17 PDT
Created attachment 370807 [details]
Patch
Comment 2 youenn fablet 2019-05-29 08:15:28 PDT
Created attachment 370851 [details]
Patch
Comment 3 youenn fablet 2019-05-29 09:17:56 PDT
<rdar://problem/47390407>
Comment 4 Eric Carlson 2019-05-29 10:29:34 PDT
Comment on attachment 370851 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=370851&action=review

> Source/WebCore/ChangeLog:9
> +        In case document gets in the background, interrupt the audio track if the audio factory allows it.

Nit: s/allows/requires/

> Source/WebCore/ChangeLog:10
> +        CoreAudioCaptureSourceIOS allows to interrupt the audio capture if the app has not the right background mode.

Nit: s/allows to interrupt the audio source/requires the audio source be interrupted/
Comment 5 youenn fablet 2019-05-29 10:34:03 PDT
Created attachment 370862 [details]
Patch for landing
Comment 6 Jon Lee 2019-05-29 11:20:00 PDT
Comment on attachment 370862 [details]
Patch for landing

View in context: https://bugs.webkit.org/attachment.cgi?id=370862&action=review

> Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.mm:2
> + * Copyright (C) 2013-2016 Apple, Inc. All rights reserved.

2019.
Comment 7 youenn fablet 2019-05-30 12:58:14 PDT
Created attachment 370967 [details]
Patch for landing
Comment 8 youenn fablet 2019-05-30 15:46:18 PDT
Created attachment 370986 [details]
Patch for landing
Comment 9 EWS Watchlist 2019-05-30 18:49:07 PDT
Comment on attachment 370986 [details]
Patch for landing

Attachment 370986 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/12333645

New failing tests:
http/wpt/cache-storage/quota-third-party.https.html
Comment 10 EWS Watchlist 2019-05-30 18:49:08 PDT
Created attachment 371013 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.14.5
Comment 11 WebKit Commit Bot 2019-05-30 21:46:13 PDT
Comment on attachment 370986 [details]
Patch for landing

Clearing flags on attachment: 370986

Committed r245946: <https://trac.webkit.org/changeset/245946>
Comment 12 WebKit Commit Bot 2019-05-30 21:46:15 PDT
All reviewed patches have been landed.  Closing bug.
Comment 13 Ryan Haddad 2019-05-31 08:24:09 PDT
Rolled out in https://trac.webkit.org/changeset/245964/webkit because it breaks the watchOS build.
Comment 14 youenn fablet 2019-05-31 09:08:00 PDT
Created attachment 371060 [details]
Fixing watchOS build
Comment 15 WebKit Commit Bot 2019-05-31 16:36:44 PDT
Comment on attachment 371060 [details]
Fixing watchOS build

Rejecting attachment 371060 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 371060, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 5000 characters of output:
=371060&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=198307&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Updating working directory
Processing patch 371060 from bug 198307.
Fetching: https://bugs.webkit.org/attachment.cgi?id=371060
Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	C	Source/WebKit/Scripts/PreferencesTemplates/WebPreferencesStoreDefaultsMap.cpp.erb => Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.mm
	A	LayoutTests/platform/ios/mediastream/audio-muted-in-background-tab-expected.txt
	A	LayoutTests/platform/ios/mediastream/audio-muted-in-background-tab.html
	M	LayoutTests/ChangeLog
	M	Source/WebCore/ChangeLog
	M	Source/WebCore/SourcesCocoa.txt
	M	Source/WebCore/WebCore.xcodeproj/project.pbxproj
	M	Source/WebCore/dom/Document.cpp
	M	Source/WebCore/page/RuntimeEnabledFeatures.h
	M	Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.cpp
	M	Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.h
	M	Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.h
	M	Source/WebCore/platform/mediastream/ios/CoreAudioCaptureSourceIOS.mm
	M	Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.h
	M	Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp
	M	Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.cpp
	M	Source/WebCore/testing/Internals.cpp
	M	Source/WebCore/testing/Internals.h
	M	Source/WebCore/testing/Internals.idl
	M	Source/WebKit/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/Source/WebKit/ChangeLog' is out of date
W: 55f594ac9f3fd5273a212606f0dceaf7be808a17 and refs/remotes/origin/master differ, using rebase:
:040000 040000 1230136b9b74a5966400983208e4e8d5921913fb 2e077803358ecc7244b1692ee4b3ea4fbcbeb3fd M	LayoutTests
:040000 040000 0a855a405d6b608d587f38b9859a41491f6d5381 5981c2cdcac9ec169150f644f3c455888f60cc7a M	Source
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	C	Source/WebKit/Scripts/PreferencesTemplates/WebPreferencesStoreDefaultsMap.cpp.erb => Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.mm
	A	LayoutTests/platform/ios/mediastream/audio-muted-in-background-tab-expected.txt
	A	LayoutTests/platform/ios/mediastream/audio-muted-in-background-tab.html
	M	LayoutTests/ChangeLog
	M	Source/WebCore/ChangeLog
	M	Source/WebCore/SourcesCocoa.txt
	M	Source/WebCore/WebCore.xcodeproj/project.pbxproj
	M	Source/WebCore/dom/Document.cpp
	M	Source/WebCore/page/RuntimeEnabledFeatures.h
	M	Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.cpp
	M	Source/WebCore/platform/mediastream/RealtimeMediaSourceCenter.h
	M	Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.h
	M	Source/WebCore/platform/mediastream/ios/CoreAudioCaptureSourceIOS.mm
	M	Source/WebCore/platform/mediastream/mac/CoreAudioCaptureSource.h
	M	Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp
	M	Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.cpp
	M	Source/WebCore/testing/Internals.cpp
	M	Source/WebCore/testing/Internals.h
	M	Source/WebCore/testing/Internals.idl
	M	Source/WebKit/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/Source/WebKit/ChangeLog' is out of date
W: 55f594ac9f3fd5273a212606f0dceaf7be808a17 and refs/remotes/origin/master differ, using rebase:
:040000 040000 1230136b9b74a5966400983208e4e8d5921913fb 2e077803358ecc7244b1692ee4b3ea4fbcbeb3fd M	LayoutTests
:040000 040000 0a855a405d6b608d587f38b9859a41491f6d5381 5981c2cdcac9ec169150f644f3c455888f60cc7a M	Source
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
From https://git.webkit.org/git/WebKit
   df5a375eb87..f44ba58dc92  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 245991 = df5a375eb87899610d78a85f85086f850824d31e
r245992 = 4c3a48aae99551cb30fd5188dc9b63f7eb71e7b8
r245993 = 102cb1d36e2df616c4ceb48a899de2302ccf68cb
r245994 = c52c2cb53e45d2c7dcb4aff67d159fc751229584
r245995 = f44ba58dc9288debbc647a54036bca56132653e7
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: https://webkit-queues.webkit.org/results/12344244
Comment 16 WebKit Commit Bot 2019-05-31 16:49:37 PDT
Comment on attachment 371060 [details]
Fixing watchOS build

Clearing flags on attachment: 371060

Committed r246002: <https://trac.webkit.org/changeset/246002>
Comment 17 WebKit Commit Bot 2019-05-31 16:49:39 PDT
All reviewed patches have been landed.  Closing bug.