Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] Private Mode screenshots possible via Tab Tray Overviews #12453

Closed
s-ankur opened this issue Jul 10, 2020 · 7 comments
Closed

[Bug] Private Mode screenshots possible via Tab Tray Overviews #12453

s-ankur opened this issue Jul 10, 2020 · 7 comments

Comments

@s-ankur
Copy link
Contributor

s-ankur commented Jul 10, 2020

Steps to reproduce

Disable screenshots in private tabs
Open a private tab
Switch to normal tab
Open tab tray
Open private mode tab tray

Expected behavior

You cant see the private mode tab overview

Actual behavior

You can see it and take a screenshot
ScreenshotUNITO-UNDERSCORE!20200710-222816!UNITO-UNDERSCORE!Firefox!UNITO-UNDERSCORE!Nightly!

Device information

  • Android device: ?
  • Fenix version: ?

┆Issue is synchronized with this Jira Task

@s-ankur s-ankur added the 🐞 bug Crashes, Something isn't working, .. label Jul 10, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Jul 10, 2020
@sheikh-azharuddin
Copy link

Works fine as expected in latest nightly... Not allowing to take screenshot in tab tray...

@s-ankur
Copy link
Contributor Author

s-ankur commented Jul 10, 2020

@sheikh-azharuddin
You need to follow the steps precisely. It is reproducible on

Nightly 200710 06:01 (Build #21920613)
AC: 50.0.20200709130100, 432c0a58c
GV: 80.0a1-20200708094217
AS: 61.0.7

@sheikh-azharuddin
Copy link

Yes true... Able to take screenshot of the preview...

@ekager ekager added the needs:UX-feedback Needs UX Feedback label Jul 13, 2020
@ekager
Copy link
Contributor

ekager commented Jul 13, 2020

UX - should we also stop screenshots while not IN PBM, but just viewing the tabs previews of PBM?

@Dunexus
Copy link

Dunexus commented Jul 14, 2020

In my opinion, when you switch to PBM tab in tab tray, it should be considered as having the browser in PBM.
All you interact with from this tab is PBM-related, private tabs, open new private tab, etc.
It is like switching from "normal" window to private window on desktop.

@AmyYLee
Copy link
Collaborator

AmyYLee commented Jul 14, 2020

UX - should we also stop screenshots while not IN PBM, but just viewing the tabs previews of PBM?

Hi,

I think this would still be a privacy concern so yes we shouldn't allow screenshots in this scenario if the user has it set to not allow screenshots in PBM.

@Diana-Rus
Copy link

Diana-Rus commented Jul 20, 2020

Hi, verified as fixed on the latest Nightly 200720 (Build 22020611) with Sony Xperia Z5 (Android 7), Samsung Galaxy S9 (Android 8) and Google Pixel 3 XL (Android 9).

Samsung GIF

20200720-112925

Note:

  • With the STR from description the issue is not reproducible
  • When the app is in recent app and the private tabs tray is opened, there won't be any information displayed.
  • In case of wanting to make a screenshot it will display an informative message: Can't take screenshot due to security policy.
  • With Sony Xperia Z5 (Android 7) the issue is not reproduce able but after putting the app in recent apps for the first time, the second time it will display the normal tab tray info, which is different outcome from Samsung Galaxy S9 (Android 8) and Google Pixel 3 XL (Android 9)-i'll create a new issue for this as soon as i found the STR as it seems that it's not reproducible all the time.

@Diana-Rus Diana-Rus added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Jul 20, 2020
@data-sync-user data-sync-user changed the title [Bug] Private Mode screenshots possible via Tab Tray Overviews FNX3-15402 ⁃ [Bug] Private Mode screenshots possible via Tab Tray Overviews Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-15402 ⁃ [Bug] Private Mode screenshots possible via Tab Tray Overviews FNX-13522 ⁃ [Bug] Private Mode screenshots possible via Tab Tray Overviews Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-13522 ⁃ [Bug] Private Mode screenshots possible via Tab Tray Overviews FNX2-15259 ⁃ [Bug] Private Mode screenshots possible via Tab Tray Overviews Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX2-15259 ⁃ [Bug] Private Mode screenshots possible via Tab Tray Overviews [Bug] Private Mode screenshots possible via Tab Tray Overviews May 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants