The default bug view has changed. See this FAQ.

'No screen' shows preview after clicking once on the device permission in the location bar

VERIFIED FIXED in Firefox 53

Status

()

Firefox
Device Permissions
P2
normal
VERIFIED FIXED
2 months ago
a month ago

People

(Reporter: pauly, Assigned: florian)

Tracking

({regression})

54 Branch
Firefox 54
regression
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +
qe-verify ?

Firefox Tracking Flags

(firefox51 unaffected, firefox52 unaffected, firefox53 verified, firefox54 verified)

Details

(Whiteboard: [fxprivacy])

Attachments

(1 attachment)

(Reporter)

Description

2 months ago
[Affected versions]:
- 54

[Affected platforms]:
- all

[Steps to reproduce]:
1. Open https://mozilla.github.io/webrtc-landing/gum_test.html
2. Click the "Screen" button
3. From the dropdown select screen to preview
4. Click once the "screen share" permission in the location bar

[Expected result]:
- No change

[Actual result]:
- 'No screen' shows preview of the active screen

[Regression range]:
- tbd

[Additional notes]:
- Screencast: https://www.screencast.com/t/tXW6CuYWQI6G
(Assignee)

Comment 1

2 months ago
Seems like an edge case of bug 1325223 that I didn't cover there.
Blocks: 1325223
(Reporter)

Updated

2 months ago
Blocks: 1004061
status-firefox53: --- → affected
(Reporter)

Updated

2 months ago
Keywords: regression
Whiteboard: [fxprivacy][triage]

Updated

2 months ago
Assignee: nobody → florian
Priority: -- → P2
Whiteboard: [fxprivacy][triage] → [fxprivacy]
(Assignee)

Comment 2

a month ago
Created attachment 8836090 [details] [diff] [review]
Patch
Attachment #8836090 - Flags: review?(jhofmann)
(Assignee)

Comment 3

a month ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=aa44bf9c62e9
Comment on attachment 8836090 [details] [diff] [review]
Patch

Review of attachment 8836090 [details] [diff] [review]:
-----------------------------------------------------------------

I haven't tried the patch locally but this looks pretty straightforward and has a test, so r=me

::: browser/base/content/test/popupNotifications/browser_popupNotification_5.js
@@ +330,5 @@
>        this.notification2.remove();
>        this.notification3.remove();
>      }
>    },
> +  // Test clicking the anchor icon.

Maybe you should mention the purpose of this test. :)
Attachment #8836090 - Flags: review?(jhofmann) → review+
(Assignee)

Comment 5

a month ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/bbcec3916fc695fa52d0dab1a3e459cf46144a1e
Bug 1336066 - avoid reshowing a persistent notification that has not been dismissed when clicking the anchor icon, r=johannh.

Updated

a month ago
Iteration: --- → 54.2 - Feb 20

Updated

a month ago
Flags: qe-verify?

Comment 6

a month ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/bbcec3916fc6
Status: NEW → RESOLVED
Last Resolved: a month ago
status-firefox54: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54

Comment 7

a month ago
Build ID: 20170214030231
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:54.0) Gecko/20100101 Firefox/54.0

Verified as fixed on on Windows 10 x 64, Mac OS X 10.11 and Ubuntu 16.04 x64 on Firefox Nightly 54.0a1, Firefox Nightly 53.0a1 and Aurora 53.0a2.
status-firefox54: fixed → verified

Updated

a month ago
Status: RESOLVED → VERIFIED
Hani, how was this verified on 53 when this hasn't been uplifted to Aurora yet?

That said, please request Aurora approval on this, Florian :)
Flags: needinfo?(hani.yacoub)
Flags: needinfo?(florian)

Comment 9

a month ago
Sorry about that, I messed up the comments with another bug that I verified on Aurora too.


I Verified this bug on Windows 10 x 64, Mac OS X 10.11 and Ubuntu 16.04 x64 on Firefox Nightly 54.0a1, as the flags indicates the version of Firefox which I tested.

Thanks.
Flags: needinfo?(hani.yacoub)
(Assignee)

Comment 10

a month ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=417b1f06fda1
(Assignee)

Comment 11

a month ago
Comment on attachment 8836090 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/Bug causing the regression]: bug 1004061
[User impact if declined]: possibly confusing state of permission prompt panels (and especially the screen sharing one with a preview) after clicking the anchor icon. Potential for leaking a video stream if there was an active screensharing preview at the time the anchor icon was clicked.
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: Not required but would be nice, steps are in comment 0.
[List of other uplifts needed for the feature/fix]: none.
[Is the change risky?]: Low.
[Why is the change risky/not risky?]: One-line code change, and covered by automated test, which seem green on the try push from comment 10.
[String changes made/needed]: none.
Flags: needinfo?(florian)
Attachment #8836090 - Flags: approval-mozilla-aurora?
Comment on attachment 8836090 [details] [diff] [review]
Patch

Fix a screensharing regression and was verified. Aurora53+.
Attachment #8836090 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Comment 13

a month ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/ce2403f3f23b
status-firefox53: affected → fixed
Flags: in-testsuite+

Comment 14

a month ago
Build ID: 20170221004019
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0

Verified as fixed on Aurora 53.0a2 (20170221004019) on Windows 10 x 64, Mac OS X 10.10 and Ubuntu 16.04 x64.
status-firefox53: fixed → verified
You need to log in before you can comment on or make changes to this bug.