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

Brave fails to open from "Open with" context menu #4795

Closed
jonathansampson opened this issue Oct 14, 2016 · 7 comments
Closed

Brave fails to open from "Open with" context menu #4795

jonathansampson opened this issue Oct 14, 2016 · 7 comments

Comments

@jonathansampson
Copy link
Collaborator

jonathansampson commented Oct 14, 2016

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

Describe the issue you encountered:
Right-clicking a file, and selecting "Open with > Brave", results in processes, but no browser.

After a minute, the spawned processes will disappear.

Expected behavior:
Right-clicking a file, and selecting "Open with > Brave", results in the file being opened in Brave.

Platform: Windows 10

Brave Version: 0.12.5

Steps to reproduce:
1. Right-click an .html file on your machine
2. Select "Open with > Brave"
3. Note that Brave does not open
4. Note that Brave.exe processes are running

@Sh1d0w
Copy link

Sh1d0w commented Oct 14, 2016

I can confirm the same bug can be reproduced on Fedora 24, so it does not seem like a Windows only issue.

@Sh1d0w Sh1d0w added bug and removed OS/Windows labels Oct 14, 2016
@neeklamy
Copy link
Contributor

This works as expected on OS X (10.11.6, Brave 0.12.4). Just to be sure I also tried quitting out of Brave, then in Finder, right clicking on an HTML document and opening with Brave – again, no problems here.

@Sh1d0w
Copy link

Sh1d0w commented Oct 14, 2016

@neeklamy The bug is reported and confirmed under 0.12.5 can you please test with latest RC?

@bsclifton
Copy link
Member

repro'ed on Windows 10

@neeklamy
Copy link
Contributor

@Sh1d0w, it works under OS X with the latest RC too, v0.12.5dev-RC1.

I tried quitting Brave, double-clicking the local HTML file, quitting right-clicking and using “Open With → Brave”. Always making sure to close the window or tab with the local file in and trying with multiple HTML files. It doesn’t repo here.

(Thanks too for editing your comment, I have to say your initial reply came across a touch snarky… Hey, I can easily move on you know…)

@Tyler-H
Copy link

Tyler-H commented Oct 15, 2016

Hi, I mentioned this bug to Jonathan the other day; my tested setups were Windows 7 64-bit Ultimate and Windows 7 64-bit Enterprise, both of them on the latest version of Brave for dev update channel (0.12.1).

Here is a short (~35 seconds) video of my experiences (failure and success) on Windows 7:
https://www.youtube.com/watch?v=ELC6qgOXNFI

I don't have access to my Windows 10 laptop to test or record an experience on there currently. I have tested with the 0.12.5.dev-RC1 for Windows7+ and the problem persists.

Perhaps it has to do with there not really being a brave.exe shell anywhere? From what I see it just exists in the user's AppData folder. So the 'thing' or 'program' I'm trying to run my HTML file in via Context Menu isn't really the Brave program?

@bbondy
Copy link
Member

bbondy commented Oct 25, 2016

User on Twitter mentioned this as a likely cause for opening local HTML files not working as well:
https://twitter.com/PermittedSoc/status/790809270690451456

@bbondy bbondy added this to the 0.12.9dev milestone Oct 25, 2016
bbondy added a commit that referenced this issue Nov 4, 2016
bbondy added a commit that referenced this issue Nov 4, 2016
bbondy added a commit that referenced this issue Nov 4, 2016
@bbondy bbondy closed this as completed in c73844c Nov 4, 2016
@bbondy bbondy modified the milestones: 0.12.8dev, 0.12.9dev Nov 4, 2016
bbondy added a commit that referenced this issue Nov 4, 2016
Fix #5352
Fix #4795

I stll need to add some automated tests for this section of code but it was also
not in testing before.

Test Plan:
- On each OS test opening .html files both whent he browser is
  started and not started.
- Type a mailto: link and http: link in Slack and click them both when
  the browser is started and not started.
- Try setting Brave as the mailto handler and not.

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

No branches or pull requests

9 participants