Can't request media permissions after navigation with active media permission prompt
Reported by
chromium...@gmail.com,
Apr 26 2016
|
||||||||||
Issue descriptionVersion : 52.0.2715.0 canary OS : Windows 7 1. Visite https://permission.site 2. Click on Video and Audio (don't allow or block them) 3. Now go to the omnibox and navigate to any website "abc.xyz" The The permission request bubble should not be displayed again.
,
Apr 26 2016
You're welcome! yeah I'm still able to reproduce this on M52.0.2717.0 - Watch please the video.
,
Apr 27 2016
Thank you for providing more feedback. Adding requester "ashejole@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 28 2016
Unable to reproduce the issue on Win7/64 bit - Version 52.0.2718.0 canary (64-bit) Steps Followed: 1. Navigate to apprtc.appspot.com which shows the permission bubble asking to Allow or Block. 2. Did not select anything. 3. Navigated to abc.xyz. Permission bubble is not seen again on abc.xyz upon being navigated to. chromium.khalil@, can you please upgrade to the latest build available and check if you still see this issue ?
,
Apr 28 2016
,
Apr 28 2016
Actually you don't have to use pprtc.appspot.com which shows the permission bubble asking to Allow or Block for Audio + Video in the same request, and that's why you're not able to reproduce. As I said in my steps in "#0 comment" you should to request Video then Audio (not Audio + Video) means you will get two permission bubbles. I've created a testcase (don't select anything): <script> navigator.webkitGetUserMedia({video: true}, function(){}, function(){}); navigator.webkitGetUserMedia({audio: true}, function(){}, function(){}); setTimeout('document.location = "https://abc.xyz";', 1000) </script>
,
Apr 28 2016
Thank you for providing more feedback. Adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 27 2016
,
Jun 27 2017
Issue has not been modified or commented on in the last 365 days, please re-open or file a new bug if this is still an issue. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 28 2017
This is a real issue, I just verified it on ChromeOS.
,
Jun 28 2017
No longer repros on Windows Canary but it's still broken. If you navigate with a media permission prompt active, it seems the tab can't request any further media permissions...
,
Jul 3 2017
(re-titling as per comment #12)
,
Jul 6 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/158a8c15d0f9aa1162c6a4583047b3a8c50046d6 commit 158a8c15d0f9aa1162c6a4583047b3a8c50046d6 Author: raymes <raymes@chromium.org> Date: Thu Jul 06 02:52:59 2017 Ensure media permission requests are correctly cancelled when permission prompts are ignored Currently when permission prompts are ignored, the PermissionManager will still hold on to the callback that was passed in to RequestPermission. |pending_requests_| is not cleaned up. This means that the callers have no way of knowing if a request was ignored. This CL changes the code so that |pending_requests_| is correctly cleaned up when a prompt is ignored. This makes media permission prompts behave correctly. BUG= 606630 Review-Url: https://codereview.chromium.org/2966963003 Cr-Commit-Position: refs/heads/master@{#484449} [modify] https://crrev.com/158a8c15d0f9aa1162c6a4583047b3a8c50046d6/chrome/browser/media/webrtc/media_stream_device_permission_context.cc [modify] https://crrev.com/158a8c15d0f9aa1162c6a4583047b3a8c50046d6/chrome/browser/media/webrtc/media_stream_device_permission_context.h [modify] https://crrev.com/158a8c15d0f9aa1162c6a4583047b3a8c50046d6/chrome/browser/permissions/permission_manager.cc [modify] https://crrev.com/158a8c15d0f9aa1162c6a4583047b3a8c50046d6/chrome/browser/permissions/permission_manager.h [modify] https://crrev.com/158a8c15d0f9aa1162c6a4583047b3a8c50046d6/chrome/browser/permissions/permission_manager_unittest.cc
,
Jul 6 2017
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by ashej...@chromium.org
, Apr 26 2016Labels: Needs-Feedback OS-Windows