Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Maximizing video, minimizing brave causes number pad to show with video. #979

Closed
gortok opened this issue Mar 16, 2019 · 6 comments
Closed
Labels
Milestone

Comments

@gortok
Copy link

gortok commented Mar 16, 2019

Description:

With a maximized video playing from YouTube, minimizing and bringing brave up via its icon causes the video to show along with the number pad.

Steps to Reproduce

1.open video on YouTube. Maximize it.
2.use lower bar swipe up gesture to minimize brave.
3.use the icon to get back to brave, note similar outcome to screenshots below.

Actual result:

CDBF991C-F110-465C-B343-41F225DCE245
39638B93-97FA-47EF-9B3E-45D501F69CC9
6ED480E5-67F7-4970-8156-1F248091D19B

Expected result:

Expect pin window to hide video.
Reproduces how often: [Easily reproduced, Intermittent Issue]
100%

Brave Version:
v1.8

Device details:
iPhone 10, iOS 12.1.4

Additional Information

@srirambv
Copy link
Contributor

Reproduced the issue. Unable to unlock browser because video overlay is blocking the passcode screen and showing only the number pad.

@srirambv srirambv added bug good first issue security priority/P2 A bad problem. We might uplift this to the next planned release. release-notes/include labels Mar 18, 2019
@srirambv srirambv modified the milestones: 1.9, 1.10 Mar 18, 2019
@kylehickinson
Copy link
Collaborator

Good place to start looking: This is likely due to the fact that a video player is a new UIWindow, and is likely placed on top of passcode entry UIWindow

@jhreis
Copy link
Contributor

jhreis commented Mar 19, 2019

Yeah, this has security concerns too then. 🤕

@jhreis jhreis added QA/Yes and removed good first issue priority/P2 A bad problem. We might uplift this to the next planned release. labels Mar 19, 2019
@tildelowengrimm
Copy link

This passcode lock feature is becoming harder and harder to maintain, and it's not even a particularly effective security boundary. We should probably drop the whole passcode lock feature.

@srirambv
Copy link
Contributor

  • Verification passed on iPhone 7+ running 1.9.2(19.04.24.20)

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented May 9, 2019

  • Verification passed on iPhone 8 running 1.9.2(19.5.8.23)
  • Verification passed on iPhone SE running 1.9.2(19.5.8.23)
  • Verification passed on iPad Mini 4 using iOS 12.2 running 1.9.2(19.5.8.23)
  • Verification passed on iPhone 6s+ using iOS 12.2 running 1.9.2(19.5.8.23)
  • Verification passed on iPad pro running 1.9.2(19.5.8.23). Verified on Twitch as YT doesn't go full screen on iPad

@jhreis jhreis added the Epic: Pincode Epic for defining issues pertaining to the iOS pincode / passcode lock. label Jan 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

9 participants