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

Cast is not disabled on toggling off media router switch in extension settings #18152

Closed
jumde opened this issue Sep 16, 2021 · 5 comments · Fixed by brave/brave-core#10116
Closed

Comments

@jumde
Copy link
Contributor

jumde commented Sep 16, 2021

Description

Toggling off media router setting in brave://settings/extensions should disable

Test Plan

Specified here: brave/brave-core#10116

Steps to Reproduce

  1. Enable cast by enabling the media router toggle in brave://settings/extensions
  2. Verify that cast functionality works
  3. Disable cast
  4. Cast should not be available in the hamburger menu

Actual result:

Cast still works

Expected result:

Cast should be disabled

Reproduces how often:

Easily reproduced

Desktop Brave version:

1.29.80 Chromium: 93.0.4577.63 (Official Build) (x86_64)

@stephendonner
Copy link

Wow; sorry for missing this!

@bridiver
Copy link
Contributor

How do you enable/disable cast? How do I verify cast functionality is working or not? The steps to repro don't provide sufficient detail.

@jumde jumde self-assigned this Sep 23, 2021
@bridiver
Copy link
Contributor

How do you enable/disable cast? How do I verify cast functionality is working or not? The steps to repro don't provide sufficient detail.

It never disables that's the bug. Go on Youtube on the latest version of Brave and a cast button will appear which should not.

Yes, I understand what the issue is, the problem is that the steps to repro are not sufficiently detailed.

@jumde
Copy link
Contributor Author

jumde commented Oct 2, 2021

@vis2x - The PR should be merged early next week.

@stephendonner
Copy link

stephendonner commented Oct 11, 2021

Verified PASSED using

Brave 1.31.81 Chromium: 95.0.4638.40 (Official Build) beta (x86_64)
Revision e3e7c76ba0284b16087cf4cf3153abfaef6470c7-refs/branch-heads/4638@{#624}
OS macOS Version 11.6 (Build 20G165)

Steps:

  1. enabled network filter for Little Snitch
  2. new profile
  3. launched Brave
  4. loaded brave://settings/extensions and verifed that cast's Media Router is disabled by default
  5. verified that SSDP requests are not sent by default
  6. toggled Media Router to On
  7. confirmed Relaunch button is visible
  8. loaded brave://flags and navigated back to brave://settings/extensions. Confirmed the Relaunch button is still visible
  9. restart browser
  10. verify cast works
  11. toggled Media Router in brave://settings/extensions to Off
  12. clicked on Relaunch
  13. verified I no longer see any cast / media-router options in YouTube or the top-level browser menus
example example example example example example
Screen Shot 2021-10-11 at 11 16 53 AM Screen Shot 2021-10-11 at 11 19 13 AM Screen Shot 2021-10-11 at 11 21 40 AM Screen Shot 2021-10-11 at 11 34 17 AM Screen Shot 2021-10-11 at 11 23 49 AM Screen Shot 2021-10-11 at 11 34 31 AM

Verification PASSED on Samsung S10+ running Android 11 using the following build:

1.31.86 Chromium: 95.0.4638.50
  • ensured that Media Router was enabled by default by visiting https://gem.cbc.ca/live/channel/ottawa and ensuring that the Cast icon was being displayed via the media player
  • ensured that tapping on the Cast button displayed a modal with all the devices that can cast
Example Example Example Example
Screenshot_20211018-165500_Brave 20211018_165541 Screenshot_20211018-165612_Brave image

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