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

Disable media-router by default #17873

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

Disable media-router by default #17873

jumde opened this issue Sep 3, 2021 · 5 comments · Fixed by brave/brave-core#9987

Comments

@jumde
Copy link
Contributor

jumde commented Sep 3, 2021

Description

Cast initiates SSDP requests by default. Cast should be disabled by default, add a setting to enable cast.

More details here: #5057 (comment)

Just a note for history: Android currently doesn't have an option and cast is always on there. We have to fix it allowing Android to cast still. That's the original issue for Android #13898

Test Plan

Specified here: brave/brave-core#9987

@kjozwiak
Copy link
Member

Removing OS/Android as this will be verified on Android via #17972.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 13, 2021

Verification passed on

Brave | 1.29.80 Chromium: 93.0.4577.63 (Official Build) (64-bit)
-- | --
Revision | ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS | Windows 10 OS Version 2009 (Build 19043.1165)

Verified that the Cast option is not available in the hamburger menu by default and SSDP requests are not initiated on start

image image

Enabled Media Router in brave://settings/extensions and confirmed Cast menu option is shown in the hamburger menu

image image

Open a youtube.com video and clicked on Cast option in the hamburger menu and confirmed there is no crash and No device found message is shown as no device are connected to cast the video
image


Verification passed on

Brave 1.29.80 Chromium: 93.0.4577.63 (Official Build) (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#10020

Verified Media Router is disabled by default in brave://settings/extensions
image

Verified that the Cast option is not available in the hamburger menu by default
image

Verified after enabling Media Router in brave://settings/extensions, Cast was added to Hamburger menu
image

Opened a youtube.com video and clicked on Cast option in the hamburger menu and confirmed there is no crash and No device found message is shown
image


Verified PASSED using

Brave 1.29.80 Chromium: 93.0.4577.63 (Official Build) (x86_64)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS macOS Version 11.5.2 (Build 20G95)

Verified test plan from brave/brave-core#10020

Verified Media Router is disabled by default in brave://settings/extensions
Screen Shot 2021-09-13 at 11 05 20 AM

Verified that the Cast option is not available in the hamburger menu by default
Screen Shot 2021-09-13 at 10 59 10 AM

Verified after enabling Media Router in brave://settings/extensions, Cast was added to Hamburger menu
Screen Shot 2021-09-13 at 10 59 47 AM

Opened https://www.youtube.com/watch?v=qpacG2GxuDA and clicked on Cast option in the hamburger menu and confirmed there is no crash, casting works
Screen Shot 2021-09-13 at 11 02 52 AM

@reyesvicente
Copy link

Encountered this just now. I just opened Brave Browser and this popped up.
Screenshot 2021-11-22 at 11 09 32 PM

@GeetaSarvadnya
Copy link

cc: @stephendonner

@stephendonner
Copy link

Encountered this just now. I just opened Brave Browser and this popped up. Screenshot 2021-11-22 at 11 09 32 PM

Please file a new issue with steps to reproduce and as much info as you can. (cc me on it, please).

I installed Lulu and have Media Router disabled and don't see anything with mediacast popping up on Lulu.

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