[webkit-reviews] review granted: [Bug 192511] [MediaStream] 'devicechange' event should not fire in frames that can't access capture devices : [Attachment 356832] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Dec 7 13:49:39 PST 2018


youenn fablet <youennf at gmail.com> has granted Eric Carlson
<eric.carlson at apple.com>'s request for review:
Bug 192511: [MediaStream] 'devicechange' event should not fire in frames that
can't access capture devices
https://bugs.webkit.org/show_bug.cgi?id=192511

Attachment 356832: Patch

https://bugs.webkit.org/attachment.cgi?id=356832&action=review




--- Comment #3 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 356832
  --> https://bugs.webkit.org/attachment.cgi?id=356832
Patch

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

> Source/WebKit/UIProcess/UserMediaPermissionRequestManagerProxy.cpp:103
> +	   auto accessState = originHasPersistentAccess ?
DeviceAccessState::PersistentAccess : DeviceAccessState::SessionAccess;

Do we need to send that information any longer?
It seems that just knowing that we have access is good enough now.

>
LayoutTests/http/tests/media/media-stream/device-change-event-in-iframe.html:6
> +    <iframe id="camera" allow="camera"
src="http://localhost:8000/media/media-stream/resources/device-change-iframe.ht
ml?2"></iframe>

For the sake of completeness, I would tend to add a frame with an
allow="microphone" as well.


More information about the webkit-reviews mailing list