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 pre-roll ads appearing in certain cases #17673

Closed
LaurenWags opened this issue Aug 24, 2021 · 4 comments · Fixed by brave/brave-core#9844
Closed

YouTube pre-roll ads appearing in certain cases #17673

LaurenWags opened this issue Aug 24, 2021 · 4 comments · Fixed by brave/brave-core#9844

Comments

@LaurenWags
Copy link
Member

Description

In the case below I'm able to reproduce getting a pre-roll add on a YouTube video every time with 1.30.x. It does not reproduce with 1.28.106 (current Release)

Steps to Reproduce

Steps:

  1. Clean profile with Nightly 1.30.44 Chromium: 93.0.4577.51
  2. Visit https://www.youtube.com/
  3. Search hot ones and click on the First We Feast channel (at top of page)
  4. Click on Videos within the First We Feast channel
  5. Choose the video below:

Screen Shot 2021-08-24 at 2 26 40 PM

Actual result:

Every time I do this on 1.30.x I get a pre-roll ad

Expected result:

No pre-roll ad like 1.28.106

Reproduces how often:

easily

Brave version (brave://version info)

Brave | 1.30.44 Chromium: 93.0.4577.51 (Official Build) nightly (x86_64)
-- | --
Revision | 762d21050e2da59930c784c09b134d0b0b148188-refs/branch-heads/4577@{#915}
OS | macOS Version 10.15.7 (Build 19H1323)

Version/Channel Information:

  • Can you reproduce this issue with the current release? no
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

cc @brave/legacy_qa @antonok-edm

@LaurenWags LaurenWags added feature/shields/adblock Blocking ads & trackers with Shields QA/Yes OS/Desktop labels Aug 24, 2021
@kjozwiak
Copy link
Member

kjozwiak commented Aug 24, 2021

Reproduced several times on Win 10 x64 using the following build while I was investigating #17669.

Brave | 1.30.43 Chromium: 93.0.4577.51 (Official Build) nightly (64-bit)
-- | --
Revision | 762d21050e2da59930c784c09b134d0b0b148188-refs/branch-heads/4577@{#915}
OS | Windows 10 OS Version 2009 (Build 19042.1165)

Once I started playing a video, I noticed that there was a re-roll add. Pressing Skip. Only seen it a handful of times but it's definitely happening.

issue

@antonok-edm
Copy link
Collaborator

Confirmed it's not caused by brave/brave-core#9673 or any of the subsequent followups - it still occurs on 1.30.22, which is before all of them.

Also, I was able to reproduce it using #brave-adblock-cosmetic-filtering-native disabled in brave://flags, so it's not specific to native cosmetic filtering.

@antonok-edm
Copy link
Collaborator

I tracked it down to brave/brave-core#9545.

Something is preventing cosmetic filtering scriptlets from being injected into the page - entering JSON.parse in the devtools console from a youtube page with a preroll add should show Proxy {length: 2, name: "parse"}, but instead it shows up as native code.

@LaurenWags
Copy link
Member Author

LaurenWags commented Sep 1, 2021

Verified passed with

Brave | 1.30.59 Chromium: 93.0.4577.58 (Official Build) beta (x86_64)
-- | --
Revision | c4410ece044414ea42fa4ba328d08195e818a99c-refs/branch-heads/4577@{#1076}
OS | macOS Version 10.15.7 (Build 19H1323)

Verified STR from description 5x, confirmed no pre-roll ads observed.
Viewed various YT videos, confirmed no pre-roll ads observed.


Verification passed on

Brave | 1.30.59 Chromium: 93.0.4577.58 (Official Build) beta (64-bit)
-- | --
Revision | c4410ece044414ea42fa4ba328d08195e818a99c-refs/branch-heads/4577@{#1076}
OS | Windows 10 OS Version 2009 (Build 19043.1165)

Verified STR from description 4x, confirmed no pre-roll ads observed.
Viewed various YT videos, confirmed no pre-roll ads observed.


Verification 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

Verified STR from description 10x, plus additional testing around the issue/area; confirmed no pre-roll ads were observed.
Viewed various YT videos, confirmed no pre-roll ads were observed.

@LaurenWags LaurenWags added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment