Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Brave Ads still show when app are in full-screen mode #5559

Closed
karenkliu opened this issue Aug 6, 2019 · 7 comments · Fixed by brave/brave-core#3380
Closed

Brave Ads still show when app are in full-screen mode #5559

karenkliu opened this issue Aug 6, 2019 · 7 comments · Fixed by brave/brave-core#3380

Comments

@karenkliu
Copy link

Description

Brave Rewards ads still show when watching a full-screen video on Youtube or VLC.

Steps to Reproduce

  1. Go to youtube.com
  2. Select a video to watch
  3. Open full-screen mode

Expected result:

Brave Rewards ads should not appear when apps are in full-screen mode.

Reproduces how often:

Easily reproduced.

Brave version (brave://version info)

Release Version 0.67.123 Chromium: 76.0.3809.87

@tmancey tmancey self-assigned this Aug 8, 2019
@tmancey tmancey assigned srirambv and unassigned tmancey Aug 8, 2019
@tmancey
Copy link
Contributor

tmancey commented Aug 8, 2019

@srirambv As discussed I cannot reproduce, could you please update ticket with your findings too. Also @karenkliu if we could arrange a screen share if the issue re-occurs so we can diagnose the problem

@srirambv
Copy link
Contributor

srirambv commented Aug 8, 2019

I am not seeing any ads when I have YT playing video in focused tab. No ads were shown when video was playing in full screen or as a standard player.

@jsecretan
Copy link

So, I can actually get this to reproduce when doing other full screen activities, like looking at a presentation. To reproduce do the following. Go to:

https://www.slideshare.net/MohammadImtiyazAhmed/the-deep-learning-technology-on-coco-framework?qid=10bdae56-cbcd-40ac-a450-1be20f377532&v=&b=&from_search=3

And select the arrows to make this full screen.

And then wait 15 seconds. Move the mouse again and an ad should appear. We should avoid ads in these cases.

@tmancey
Copy link
Contributor

tmancey commented Aug 9, 2019

@jsecretan but we would need to classify a page on whether a user is working, casual browsing, shopping etc. Right now we can only detect if video or audio is playing. And only block ad notifications if video is playing in the focussed tab.

@tmancey tmancey added the QA/Yes label Aug 14, 2019
@karenkliu
Copy link
Author

Okay, I found some examples:

  1. Crunchyroll is an American distributor, publisher, licensing company and international online community focused on video streaming East Asian media including anime, manga, drama, music, electronic entertainment, and content.

crunchyroll

  1. Putlocker is an online index of hosted files and a website used for streaming entertainment media such as films and television series.

torrent streaming

Both of these sites showed Brave Ads while the video was playing.

@karenkliu karenkliu changed the title Brave Rewards ads still show when app are in full-screen mode Brave Ads still show when app are in full-screen mode Aug 21, 2019
@jsecretan jsecretan added the priority/P3 The next thing for us to work on. It'll ride the trains. label Sep 5, 2019
@tmancey tmancey self-assigned this Sep 6, 2019
@tmancey tmancey added this to the 0.72.x - Nightly milestone Sep 6, 2019
@rebron rebron removed this from the 0.71.x - Dev milestone Sep 18, 2019
@rebron
Copy link
Collaborator

rebron commented Sep 18, 2019

Clearing milestone.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 18, 2019

Verification passed on

Brave 1.1.4 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Verified passed with

Brave 1.1.5 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS macOS Version 10.13.6 (Build 17G5019)

Verification passed on

Brave 1.1.4 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Ubuntu 18.04 LTS
  • Verified test plan from Brave Ads still show when app are in full-screen mode brave-core#3380
  • Verified no ads displayed when Brave was in full screen mode
  • Verified ads were displayed when Brave was not in full screen mode
  • Verified no ads were displayed when media was playing in a tab in full screen mode (used Netflix, brave.com, cnet.com, Hulu)
  • Verified no ads were displayed when media was playing in a tab which was not in full screen mode (used Netflix, brave.com, cnet.com, Hulu)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants