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

Force network requests from YouTube to be blocked in aggressive mode #17607

Closed
antonok-edm opened this issue Aug 20, 2021 · 4 comments · Fixed by brave/brave-core#9808
Closed

Comments

@antonok-edm
Copy link
Collaborator

YouTube sometimes breaks spectacularly when not kept in sync with uBlock Origin's upstream filter changes. We should keep our behavior on *.youtube.com as similar as possible, meaning kept in aggressive mode at all times.

@antonok-edm antonok-edm added feature/shields/adblock Blocking ads & trackers with Shields OS/Android Fixes related to Android browser functionality OS/Desktop labels Aug 20, 2021
@antonok-edm antonok-edm self-assigned this Aug 20, 2021
@antonok-edm
Copy link
Collaborator Author

Leaving this as release-notes/exclude because it's a tweak to the bigger change in #17366

@Ayern12
Copy link

Ayern12 commented Aug 23, 2021 via email

@antonok-edm antonok-edm added this to the 1.30.x - Nightly milestone Aug 23, 2021
@stephendonner
Copy link

stephendonner commented Aug 23, 2021

Verified PASSED using

Brave 1.30.42 Chromium: 93.0.4577.51 (Official Build) nightly (x86_64)
Revision 762d21050e2da59930c784c09b134d0b0b148188-refs/branch-heads/4577@{#915}
OS macOS Version 11.5.2 (Build 20G95)

YouTube.com Steps:

  1. Set the brave-adblock-default-1p-blocking flag to Disabled in brave://flags.
  2. Visited youtube.com and clicked on a video.
  3. Ensured no pre-roll ads appear.
  4. Verified that there are blocked requests in the Shields panel, and that at least one is from youtube.com or one of its subdomains.
example example
Screen Shot 2021-08-23 at 3 00 32 PM Screen Shot 2021-08-23 at 2 51 32 PM

m.YouTube.com Steps:

  1. Set the brave-adblock-default-1p-blocking flag to Disabled in brave://flags.
  2. Visited https://m.youtube.com/?persist_app=1&app=m and clicked on a video.
  3. Ensured no pre-roll ads appear.
  4. Verified that there are blocked requests in the Shields panel, and that at least one is from youtube.com or one of its subdomains.
example example
Screen Shot 2021-08-23 at 2 50 12 PM Screen Shot 2021-08-23 at 2 54 33 PM


Verified PASSED using

Brave 1.30.63 Chromium: 93.0.4577.63 (Official Build) beta (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Linux

YouTube.com Steps:

  1. Set the brave-adblock-default-1p-blocking flag to Disabled in brave://flags.
  2. Visited youtube.com and clicked on a video.
  3. Ensured no pre-roll ads appear.
  4. Verified that there are blocked requests in the Shields panel, and that at least one is from youtube.com or one of its subdomains.
example example
Screen Shot 2021-09-03 at 11 02 20 AM Screen Shot 2021-09-03 at 11 02 56 AM

m.YouTube.com Steps:

  1. Set the brave-adblock-default-1p-blocking flag to Disabled in brave://flags.
  2. Visited https://m.youtube.com/?persist_app=1&app=m and clicked on a video.
  3. Ensured no pre-roll ads appear.
  4. Verified that there are blocked requests in the Shields panel, and that at least one is from youtube.com or one of its subdomains.

example

Screen Shot 2021-09-03 at 11 05 31 AM


Verification passed on

Brave | 1.30.65 Chromium: 93.0.4577.63 (Official Build) beta (64-bit)
-- | --
Revision | ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS | Windows 10 OS Version 2009 (Build 19043.1165)

youtube.com

Verified that there are blocked requests in the Shields panel and that at least one is from youtube.com or one of its subdomains.

example example
image image

m.youtube.com

Verified that there are blocked requests in the Shields panel and that at least one is from youtube.com or one of its subdomains.

example example
image image

@srirambv
Copy link
Contributor

Verification passed on Oppo Reno 5 with Android 11 running 1.30.76 x64 beta build

  • Verified test plan from brave-core#9808
  • Verified disabling flag doesn't show pre-roll ads on youtube.com or m.youtube.com
image image image

Verification passed on Samsung Tab A with Android 10 running 1.30.76 x64 beta build

  • Verified test plan from brave-core#9808
  • Verified disabling flag doesn't show pre-roll ads on youtube.com or m.youtube.com
image image image

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