Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Autoplay notification should be dismissed if user ignores notification and continues stream #9143

Closed
srirambv opened this issue May 30, 2017 · 2 comments · Fixed by #10178
Closed

Comments

@srirambv
Copy link
Collaborator

srirambv commented May 30, 2017

Test plan

  1. Set Auto play to Always Ask
  2. Visit http://vimeo.com and play a video, notification bar should be shown
  3. Click on play button to continue media stream
  4. Autoplay notification bar is dismissed

Original issue description

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Autoplay notification should be dismissed if user ignores and choose to play

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    All

  • Brave Version (revision SHA):
    Brave 0.16.0
    rev be76cd3
    Muon 3.2.101

  • Steps to reproduce:

    1. Set Auto play to Always Ask
    2. Visit http://vimeo.com and play a video, notification bar should be shown
    3. Click on play button to continue media stream, autoplay notification bar is not dismissed
  • Actual result:
    Autoplay notification is not dismissed when user ignores and clicks play

  • Expected result:
    Should dismiss notification bar for that video stream when user ignores it

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    No

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:

@srirambv srirambv added bug/good-first-bug feature/autoplay polish Nice to have — usually related to front-end/visual tasks. suggestion labels May 30, 2017
darkdh added a commit that referenced this issue Jul 27, 2017
fixes #9143
fixes #9008
fixes #9171

Auditors: @bsclifton, @bridiver, @cezaraugusto

Test Plan:
Covered by automatic test

Specific STR for #9171:
1. Turn on autoplay to `always ask` and make sure there is no any allow
autoplay permissions
2. Go to https://youtu.be/g_6yBZKj-eo and don't click on notification
bar
3. Open another tab and go to
https://www.w3schools.com/tags/tryit.asp?filename=tryhtml5_video_autoplay
and still don't click on notification bar
4. Go back the youtube tab you will find out the notification bar is non
clickable
5. Go to the w3schools tab and it will also be non clickable
@darkdh darkdh mentioned this issue Jul 27, 2017
8 tasks
@darkdh darkdh added this to the 0.20.x (Nightly Channel) milestone Jul 27, 2017
darkdh added a commit that referenced this issue Jul 31, 2017
fixes #9143
fixes #9008
fixes #9171

Auditors: @bsclifton, @bridiver, @cezaraugusto

Test Plan:
Covered by automatic test

Specific STR for #9171:
1. Turn on autoplay to `always ask` and make sure there is no any allow
autoplay permissions
2. Go to https://youtu.be/g_6yBZKj-eo and don't click on notification
bar
3. Open another tab and go to
https://www.w3schools.com/tags/tryit.asp?filename=tryhtml5_video_autoplay
and still don't click on notification bar
4. Go back the youtube tab you will find out the notification bar is non
clickable
5. Go to the w3schools tab and it will also be non clickable
@darkdh darkdh modified the milestones: 0.20.x (Beta Channel), 0.19.x Hotfix 11 (Release Channel) Jan 4, 2018
@kjozwiak
Copy link
Member

kjozwiak commented Jan 4, 2018

Clearing flags as this was tested against 0.20.x, needs to be re-tested against 0.19.x.

@srirambv
Copy link
Collaborator Author

srirambv commented Jan 5, 2018

Verified on 0.19.126. Clicking play though notification works and video playback starts.

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

Successfully merging a pull request may close this issue.

6 participants