RESOLVED FIXED 180699
getUserMedia is resolving before the document knows it is capturing 
https://bugs.webkit.org/show_bug.cgi?id=180699
Summary getUserMedia is resolving before the document knows it is capturing
youenn fablet
Reported 2017-12-12 09:13:27 PST
Sometimes getUserMedia promise will be resolved and autoplay will not be disabled as the promise resolves before the stream is producing data and the document knows it is capturing. We may want to delay promise resolution until the stream is ready to produce data.
Attachments
Patch (9.73 KB, patch)
2017-12-12 10:09 PST, youenn fablet 		no flags
DetailsFormatted DiffDiff
Patch (21.88 KB, patch)
2017-12-12 11:19 PST, youenn fablet 		no flags
DetailsFormatted DiffDiff
Rebasing (16.16 KB, patch)
2017-12-12 13:38 PST, youenn fablet 		no flags
DetailsFormatted DiffDiff
Patch for landing (16.16 KB, patch)
2017-12-12 15:17 PST, youenn fablet 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2017-12-12 10:09:51 PST
Created attachment 329119 [details] Patch
youenn fablet
Comment 2 2017-12-12 11:19:12 PST
Created attachment 329130 [details] Patch
youenn fablet
Comment 3 2017-12-12 13:38:35 PST
Created attachment 329148 [details] Rebasing
Eric Carlson
Comment 4 2017-12-12 15:12:46 PST
Comment on attachment 329148 [details] Rebasing View in context: https://bugs.webkit.org/attachment.cgi?id=329148&action=review > Source/WebCore/Modules/mediastream/UserMediaRequest.cpp:260 > + { Nit: indentation.
youenn fablet
Comment 5 2017-12-12 15:17:33 PST
Created attachment 329166 [details] Patch for landing
WebKit Commit Bot
Comment 6 2017-12-12 17:39:39 PST
Comment on attachment 329166 [details] Patch for landing Clearing flags on attachment: 329166 Committed r225828: <https://trac.webkit.org/changeset/225828>
WebKit Commit Bot
Comment 7 2017-12-12 17:39:41 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2017-12-12 17:40:32 PST
<rdar://problem/36010366>
Note You need to log in before you can comment on or make changes to this bug.