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

Ads not shown if a background tab is playing video #3779

Closed
srirambv opened this issue Mar 19, 2019 · 2 comments · Fixed by brave/brave-core#2575
Closed

Ads not shown if a background tab is playing video #3779

srirambv opened this issue Mar 19, 2019 · 2 comments · Fixed by brave/brave-core#2575

Comments

@srirambv
Copy link
Contributor

Description

Ads not shown if a background tab is playing video

Steps to Reproduce

  1. Enable rewards/ads on a new profile
  2. Play a YT video in a background tab
  3. Open a new tab and complete criteria to trigger ad
  4. Console shows error saying Notification not made: Media playing in browser

Actual result:

[1:1:0319/101004.376191:INFO:ads_impl.cc(628)] Notification not made: Media playing in browser

Expected result:

Ad should be shown

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.63.14 Chromium: 73.0.3683.75 (Official Build) dev(64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Linux

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?

Website problems only:

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

Additional Information

cc: @brave/legacy_qa @tmancey @jsecretan @mandar-brave

@kjozwiak
Copy link
Member

kjozwiak commented Mar 20, 2019

Seeing the same thing on macOS 10.14.3 x64 using the following build:

Brave 0.63.14 Chromium: 73.0.3683.75 (Official Build) dev(64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Mac OS X

Example:

[7814:775:0319/204849.718123:INFO:ads_impl.cc(209)] Browser state changed to idle
[7814:775:0319/204902.730308:INFO:ads_impl.cc(216)] Browser state changed to unidle
[7814:775:0319/204902.731913:INFO:ads_impl.cc(628)] Notification not made: Media playing in browser

Nice find @srirambv 👍

@jsecretan jsecretan added the priority/P3 The next thing for us to work on. It'll ride the trains. label Mar 22, 2019
@tmancey tmancey added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Jun 3, 2019
@tmancey tmancey self-assigned this Jun 3, 2019
@tmancey tmancey modified the milestone: 0.68.x - Nightly Jun 3, 2019
tmancey added a commit to brave/brave-core that referenced this issue Jun 4, 2019
@tmancey tmancey added this to the 0.68.x - Nightly milestone Jun 4, 2019
@LaurenWags
Copy link
Member

LaurenWags commented Aug 5, 2019

Verified passed on

Brave 0.68.118 Chromium: 76.0.3809.87 (Official Build) beta (64-bit)
Revision 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Mac OS X
  • Verified test plan from Fix ads not shown if a background tab is playing video brave-core#2575
  • When media (YT, Netflix, rainymood.com) is playing on the non-active (blurred) tab and brave is in the foreground, I am able to get notifications for ads.
  • When media (YT, Netflix) is playing in the active tab, I see [23278:775:0805/161334.958972:INFO:ads_impl.cc(641)] Notification not made: Media playing in browser in the logs.

Verification passed on

Brave 0.68.118 Chromium: 76.0.3809.87 (Official Build) beta (64-bit)
Revision 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Ubuntu 18.04 LTS

Verification passed on

Brave 0.68.129 Chromium: 76.0.3809.100 (Official Build) (64-bit)
Revision ed9d447d30203dc5069e540f05079e493fc1c132-refs/branch-heads/3809@{#990}
OS Windows 10 OS Version 1803 (Build 17134.523)

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.

7 participants