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

YouTube ad's are not blocked sometimes #242

Closed
srirambv opened this issue May 29, 2018 · 7 comments
Closed

YouTube ad's are not blocked sometimes #242

srirambv opened this issue May 29, 2018 · 7 comments

Comments

@srirambv
Copy link
Contributor

Description

YouTube ad's are not blocked sometimes

Steps to Reproduce

  1. Build from source
  2. Watch YouTube videos
  3. Some ads leak through either as pre-roll or in between stream

Actual result:

image

Expected result:

All YouTube ads should be blocked

Reproduces how often:

Easy(just need to keep watching YT)

Brave version (about:brave info)

2e9c470

@srirambv srirambv added release/blocking feature/shields/adblock Blocking ads & trackers with Shields labels May 29, 2018
@bbondy
Copy link
Member

bbondy commented Jun 2, 2018

Does it work differently from browser-laptop?

@srirambv
Copy link
Contributor Author

srirambv commented Jun 2, 2018

Yes. No pre-roll ad or instream ads are seen on browser-laptop. Once it starts doesn't stop. Have to clear browsing history and start over for it to start blocking ads

@bbondy bbondy self-assigned this Jun 6, 2018
@bbondy bbondy added this to the Milestone 3: June-July milestone Jun 6, 2018
@bbondy bbondy assigned emerick and unassigned bbondy Jun 13, 2018
@emerick
Copy link
Contributor

emerick commented Jun 13, 2018

@srirambv Could you give this another try please? I believe an erroneous file was on our S3 server. If by chance you continue to see the problem, it would be helpful to capture the chrome_debug.log and a screenshot of chrome://components/ as well if possible. Thanks!

@srirambv
Copy link
Contributor Author

@emerick Not seeing on a clean profile built from 332140d. Closing the issue for now. If it reproduces will capture the details and reopen the issue.

@bbondy
Copy link
Member

bbondy commented Jun 14, 2018

🎉

@emerick
Copy link
Contributor

emerick commented Jun 14, 2018

Thanks @srirambv!

@srirambv
Copy link
Contributor Author

srirambv commented Sep 25, 2018

Verification Passed on

Brave 0.55.6 Chromium: 70.0.3538.16 (Official Build) dev (64-bit)
Revision 16ed95b41bb05e565b11fb66ac33c660b721f778-refs/branch-heads/3538@{#306}
OS Linux
  • No pre-roll ads seen on YT before playback
  • No in video ads seen on YT while playback

Verification passes on

Brave 0.55.6 Chromium: 70.0.3538.16 (Official Build) dev (64-bit)
Revision 16ed95b41bb05e565b11fb66ac33c660b721f778-refs/branch-heads/3538@{#306}
OS Windows 7

Went through verification using the following build under macOS 10.13.6 x64 - PASSED

Brave 0.55.10 Chromium: 70.0.3538.22 (Official Build) beta(64-bit)
Revision ac9418ba9c3bd7f6baaffa0b055dfe147e0f8364-refs/branch-heads/3538@{#468}
OS Mac OS X

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

No branches or pull requests

6 participants