Wrong message is displayed when trying to allow permanent access to your screen or window

VERIFIED FIXED in Firefox 55

Status

()

Firefox
Site Identity and Permission Panels
P2
normal
VERIFIED FIXED
5 months ago
3 months ago

People

(Reporter: Hani Yacoub, Assigned: nhnt11)

Tracking

(Blocks: 1 bug)

54 Branch
Firefox 55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 wontfix, firefox54 wontfix, firefox55 verified)

Details

(Whiteboard: [fxprivacy])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

5 months ago
Created attachment 8837666 [details]
Screen_permanent_access.png

[Affected versions]: 
Nightly 54.0a1

[Affected platforms]:
All platforms: Ubuntu 16.04 x64, Windows 10 x 64, Mac OS X 10.11

[Steps to reproduce]:
1. Go to https://people-mozilla.org/~fqueze2/webrtc/
2. Click on "Screen".
3. Check "Remember this decision".

[Expected result]:
"%S can not allow permanent access to your screen." message is displayed.

[Actual result]:
"Nightly can not allow permanent access to your screen without asking which one to share." message is displayed.
(Reporter)

Updated

5 months ago
Blocks: 1188147
Whiteboard: [fxprivacy] [triage]
Florian do you have any thoughts here?
Flags: needinfo?(florian)
Priority: -- → P2
Whiteboard: [fxprivacy] [triage] → [fxprivacy]
I don't remember the outcome of the discussion we had with Michelle about this string. Either of the strings mentioned in comment 0 seem technically accurate, but I guess the shorter "can not allow permanent access to your screen." may be less confusing to users.
Flags: needinfo?(florian)
(Reporter)

Comment 3

5 months ago
This bug was logged based on comment https://bugzilla.mozilla.org/show_bug.cgi?id=1319112#c22.
(Assignee)

Updated

5 months ago
Assignee: nobody → nhnt11
Blocks: 1319112
Comment hidden (mozreview-request)
Comment on attachment 8846636 [details]
Bug 1339848 - Simplify the string used to indicate that permanent access to the screen cannot be granted.

https://reviewboard.mozilla.org/r/119656/#review121516
Attachment #8846636 - Flags: review?(florian) → review+

Comment 6

4 months ago
Pushed by nhnt11@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a77ebcd6f910
Simplify the string used to indicate that permanent access to the screen cannot be granted. r=florian
Status: NEW → ASSIGNED

Comment 7

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a77ebcd6f910
Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
(Reporter)

Comment 8

4 months ago
Build ID: 20170317030204
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:55.0) Gecko/20100101 Firefox/55.0

Verified as fixed on Firefox Nightly 55.0a1 on Windows 10 x 64, Mac OS X 10.12 and Ubuntu 16.04 x64.
Status: RESOLVED → VERIFIED
status-firefox55: fixed → verified
(Reporter)

Comment 9

4 months ago
Shouldn't this be uplifted to Aurora?
Flags: needinfo?(jhofmann)
Flags: needinfo?(bmo)
String change :/
status-firefox54: affected → wontfix
Flags: needinfo?(jhofmann)
Flags: needinfo?(bmo)
(Reporter)

Updated

4 months ago
status-firefox53: --- → affected
status-firefox53: affected → wontfix
You need to log in before you can comment on or make changes to this bug.