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

Muted tab state lost after tear off #11304

Closed
kevinlawler opened this issue Oct 5, 2017 · 2 comments
Closed

Muted tab state lost after tear off #11304

kevinlawler opened this issue Oct 5, 2017 · 2 comments
Assignees
@darkdh
Labels
duplicate Issue has already been reported feature/tear-off-tabs QA/checked-Linux QA/checked-macOS QA/checked-Win64 release-notes/exclude
Milestone
0.22.x Release 3 ...

Comments

@kevinlawler
Copy link
Contributor

kevinlawler commented Oct 5, 2017 •
edited
Loading

Description

Muting a tab and then tearing it off causes the tab to start playing sound again. The image depicting the speaker state falsely indicates the tab is still muted.

Steps to Reproduce

  1. Go to and address that plays audio, for example https://apple.news/AjVbOd78-SZSt_wI5JbhVqg

screen shot 2017-10-05 at 11 17 15 am

  1. Mute the tab's sound using the speaker icon at the top

screen shot 2017-10-05 at 11 17 41 am

  1. Drag the tab to tear it off the current browser window, resulting in a new window

screen shot 2017-10-05 at 11 18 13 am

Actual result:

copy

Expected result:

Sound should not be playing.

Reproduces how often:

100% (macOS)

Brave Version

Brave: 0.18.36
rev: 7ab85e9
Muon: 4.3.22
libchromiumcontent: 61.0.3163.79
V8: 6.1.534.32
Node.js: 7.9.0
Update Channel: dev
OS Platform: macOS
OS Release: 16.7.0
OS Architecture: x64

Reproducible on current live release:

Yes

Additional Information

Tested on macOS using bluetooth headset.
@kevinlawler kevinlawler changed the title muted tab state lost after tear off Muted tab state lost after tear off Oct 5, 2017
@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@petemill petemill modified the milestones: Triage Backlog, 0.22.x Release 3 (Beta channel) May 7, 2018
@petemill
Copy link
Member

petemill commented May 7, 2018

This will be fixed with next muon build for 0.22.x-release3

@petemill petemill closed this as completed May 7, 2018
This was referenced May 8, 2018
Closed
Closed
Closed
@btlechowski
Copy link
Contributor

btlechowski commented May 8, 2018 •
edited by srirambv
Loading

Duplicated by #13979

Verified on Ubuntu 17.10 x64

  • 0.22.703 903b8d0
  • libchromiumcontent 66.0.3359.139
  • muon: 6.0.8

Verified on macOS 10.13.3 x64 using the following build:

  • 0.22.703 903b8d0
  • libchromiumcontent: 66.0.3359.139
  • muon: 6.0.8

Verified on Windows x64

  • 0.22.703 903b8d0
  • libchromiumcontent 66.0.3359.139
  • muon: 6.0.8

@kjozwiak kjozwiak mentioned this issue May 16, 2018
Closed
@kjozwiak kjozwiak added duplicate Issue has already been reported release-notes/exclude and removed release-notes/include labels May 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@darkdh darkdh

Labels
duplicate Issue has already been reported feature/tear-off-tabs QA/checked-Linux QA/checked-macOS QA/checked-Win64 release-notes/exclude
Projects
None yet
Milestone
0.22.x Release 3 (Release Channel)
Development

No branches or pull requests
8 participants
@kevinlawler @petemill @kjozwiak @luixxiul @bsclifton @darkdh @srirambv @btlechowski