Bug 197566

Summary: [iOS] set the default maximum camera count to 1 for enumerateDevices
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebRTCAssignee: youenn fablet <youennf>
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 for landing none

youenn fablet
Reported 2019-05-03 09:52:04 PDT
[iOS] set the default maximum camera count to 1 for enumerateDevices
Attachments
Patch (4.02 KB, patch)
2019-05-03 10:22 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Patch for landing (3.95 KB, patch)
2019-05-03 15:20 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2019-05-03 10:22:04 PDT
Created attachment 368946 [details] Patch
Eric Carlson
Comment 2 2019-05-03 10:53:53 PDT
Comment on attachment 368946 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=368946&action=review > Source/WebKit/ChangeLog:8 > + The previous number for iOS was 2 as there are a front and back camera. Nit: s/there are a front/there is a front/
youenn fablet
Comment 3 2019-05-03 15:20:40 PDT
Created attachment 369003 [details] Patch for landing
WebKit Commit Bot
Comment 4 2019-05-03 16:12:51 PDT
The commit-queue encountered the following flaky tests while processing attachment 369003 [details]: fetch/fetch-worker-crash.html bug 187257 (author: youennf@gmail.com) The commit-queue is continuing to process your patch.
WebKit Commit Bot
Comment 5 2019-05-03 16:13:42 PDT
Comment on attachment 369003 [details] Patch for landing Clearing flags on attachment: 369003 Committed r244935: <https://trac.webkit.org/changeset/244935>
WebKit Commit Bot
Comment 6 2019-05-03 16:13:44 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 7 2019-05-03 16:14:19 PDT
<rdar://problem/50461108>
Note You need to log in before you can comment on or make changes to this bug.