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 (2) #17855

Closed
Mugurell opened this issue Feb 5, 2021 · 11 comments
Closed

[Bug] Private Mode screenshots possible via Tab Tray Overviews (2) #17855

Mugurell opened this issue Feb 5, 2021 · 11 comments
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:PrivateBrowsing S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist

Comments

@Mugurell
Copy link
Contributor

Mugurell commented Feb 5, 2021

Similar to #17854 and #12453 one can unknowingly allow the list of opened private tabs screenshotted / recorded.

Steps to reproduce

  • Have Settings -> Private browsing -> Allow screenshots in private mode disabled
  • have a few normal tabs and a few private tabs opened
  • open one normal tab / got to normal tabs homescreen
  • open tabs tray -> choose to create a new private tab
  • try taking screenshots of all the tabs shown in tabs tray, incl private ones.

Expected behavior

A message from Android saying that screenshots are not allowed

Actual behavior

Screenshots are possible

To note

When putting the app in background and resuming the FLAG_SECURE gets set (not all the times) so now one cannot take record the screen while in private mode.

Video showing these cases (use the android message / screenshot as indicators of when screenshots are blocked or not):

OpenedPrivateTabs.mov

┆Issue is synchronized with this Jira Task

@Mugurell Mugurell added 🐞 bug Crashes, Something isn't working, .. Feature:PrivateBrowsing labels Feb 5, 2021
@github-actions github-actions bot added the needs:triage Issue needs triage label Feb 5, 2021
@Mugurell Mugurell added S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist and removed needs:triage Issue needs triage labels Feb 5, 2021
@ekager ekager added the eng:qa:needed QA Needed label Feb 5, 2021
@lobontiumira
Copy link

lobontiumira commented Feb 8, 2021

This is not fixed entirely, I was able to take a screenshot in private browsing when the keyboard is trigerred, as it can be seen in the video below:

screenshot.mp4

Environment:

  • 2/8 Nightly build
  • Samsung Galaxy Note 8 (Android 9)

@lobontiumira lobontiumira removed the eng:qa:needed QA Needed label Feb 8, 2021
@ekager ekager self-assigned this Feb 8, 2021
@lobontiumira
Copy link

Reproducible on Beta 86.0.0-beta.5, and 2/15 Nightly with Sony Xperia Z5 Premium (Android 7.1.1).

@lobontiumira
Copy link

Reproducible on RC 86.1.0 with Sony Xperia Z5 Premium (Android 7.1.1).

@ekager ekager removed their assignment Feb 23, 2021
@lobontiumira
Copy link

This issue still reproduces on the 3/22 Nightly build with Samsung Galaxy Note 8 (Android 9).
I was able to take a screenshot in private browsing when the keyboard is trigerred.

@lobontiumira
Copy link

Reproducible on Beta 88.0.0-beta.6 with OnePlus 5T (Android 10).

@lobontiumira
Copy link

Reproducible on RC 88.1.0 with Xiaomi Mi 8 Lite (Android 9).

@lobontiumira
Copy link

Reproducible on RC 88.1.4 with Samsung Galaxy Note 8 (Android 9).

@lobontiumira
Copy link

Reproducible on Beta 90.0.0-beta.5 with Google Pixel (Android 10).

@Nickoriginal
Copy link

#17854 (comment)

@lobontiumira
Copy link

Reproducible on 90.1.0 RC build with OnePlus 5T (Android 10).

codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue Jul 8, 2021
@SoftVision-LorandJanos
Copy link

Verified as fixed in the latest Nightly (91.0a1 2021-07-09T05:07:18.274).
Devices used: Motorola Nexus 6 (Android 7.1.1), Samsung Galaxy Tab 3 (Android 9), Google Pixel 4 (Android 11) and Huawei P9 Lite (Android 7.0).

czlucius pushed a commit to czlucius/fenix that referenced this issue Aug 20, 2021
czlucius pushed a commit to czlucius/fenix that referenced this issue Aug 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:PrivateBrowsing S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Projects
None yet
Development

No branches or pull requests

7 participants