[webkit-reviews] review granted: [Bug 178754] [MediaStream] Clear cached gUM prompt state : [Attachment 324860] Rebased and updated patch.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 25 14:50:00 PDT 2017


youenn fablet <youennf at gmail.com> has granted Eric Carlson
<eric.carlson at apple.com>'s request for review:
Bug 178754: [MediaStream] Clear cached gUM prompt state
https://bugs.webkit.org/show_bug.cgi?id=178754

Attachment 324860: Rebased and updated patch.

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




--- Comment #6 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 324860
  --> https://bugs.webkit.org/attachment.cgi?id=324860
Rebased and updated patch.

LGTM.

I am still not sure whether we should not remove the denied requests decisions
after some timeouts.
The timer will only fire if user granted either audio or video.
So there will be a prompt for one or the other.
Then, why not allowing the application to request both?


More information about the webkit-reviews mailing list