1289755 - Permission doorhangers shouldn't open if you already made a decision on that permission

Status: NEW

(Firefox :: Site Permissions, defect, P3)

Description

[Paul Silaghi, QA [:pauly]]

[Affected versions]:
- 50.0a1 (2016-07-26)

[Affected platforms]:
- all

[Steps to reproduce]:
1. Open http://permission.site/
2. Click on the "Microphone" and "Location" form
3. While the permission pop-ups remain open in the background, open the Control Center/More information/Permissions
4. Set the "microphone" and "location" permissions to "block" and close the permissions window
5. In the location bar click on the microphone and location icons that aren't crossed out
6. Click "Share"

[Actual results]:
- Still able to share the microphone and location even if the permissions are blocked

[Expected results]:
- Shouldn't be able to open permission doorhangers if you already made a decision on that permission

[Regression range]:
- old bug, repro on FX 10

Comment 1

[Paul Silaghi, QA [:pauly]]

Attachment: permissions.png

Comment 2

[Mark Hammond [:markh] [:mhammond]]

The indicators in the URL bar allow you to grant permissions later if you choose. They seem to clearly show the permission is blocked while offering a target to change the current settings. Why is this considered a bug?

Comment 3

[Paul Silaghi, QA [:pauly]]

(In reply to Mark Hammond [:markh] from comment #2)
> The indicators in the URL bar allow you to grant permissions later if you
> choose.
IMO that is not correct, they defy the purpose of blocking.

Comment 4

[Panos Astithas (he/him) [:past] (please ni?)]

The identity block icons shouldn't be simultaneously in a "request" and a "blocked" state, because it is too confusing. Ideally modifying the permissions from the Page Info dialog (which is what comment 0 refers to) would also respond to the pending permission requests instead of letting them linger.

Comment 5

[Mark Hammond [:markh] [:mhammond]]

I missed the fact that the icons are shown twice in the identity block, and agree that's confusing. Comment 0 also refers to doorhangers re-opening, which although not shown in the screenshot, is also bad.  

I *do* think it's fine that the "blocked" indicators (only) remain in the identity block, and what I mistakenly thought the bug was about. Sorry for the noise.

Comment 6

[David Lawrence [:dkl]]

Changing qe-verify? to qe-verify+.