Closed Bug 1251222 Opened 8 years ago Closed 8 years ago

Hide context menu in private browsing tabs when in the app switcher

Categories

(Firefox for iOS :: General, defect)

Product:
Firefox for iOS
Component:
General
Platform:
Other
iOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Tracking Flags:
Tracking Status
fxios-v4.0 --- affected
fxios-v5.0 --- verified
fxios + ---

People

(Reporter: TeoVermesan, Assigned: tlacroix, Mentored)

References

Details

(Keywords: privacy)

Attachments

(2 files)

Screen Shot 2016-02-25 at 11.21.46 AM.png
147.78 KB, image/png
Details
https://github.com/mozilla/firefox-ios/pull/1794
48 bytes, text/x-github-pull-request
sleroux
: review+
Details | Review 
Steps to reproduce:
1. Go to private browsing
2. Go to news.google.com
3. Long tap on an article to invoke context menu
4. Go to the app switcher

Actual results:
- The page is grey, but the context menu is still displayed

Expected results:
- Hide context menu in private browsing tabs when in the app switcher when the user was previously on the tab tray
Potential URL leak
tracking-fxios: --- → ?
Keywords: privacy
Mentor: sleroux
See Also: → 1257219
Assignee: nobody → tlacroix
Comment on attachment 8750796 [details] [review]
https://github.com/mozilla/firefox-ios/pull/1794

Looks good! I like that this also applies to the various popovers like share menu as well.
Attachment #8750796 - Flags: review?(sleroux) → review+
master 1649ceb86c5f54deb82a74bb46950bafd35ff2c0
Status: NEW → RESOLVED
Closed: 8 years ago
status-fxios-v4.0: --- → affected
status-fxios-v5.0: --- → fixed
Resolution: --- → FIXED
Verifying as fixed on master 1649ceb86
Status: RESOLVED → VERIFIED
status-fxios-v5.0: fixed → verified
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: