Open html file with drag and drop

[Sh1d0w]

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

Describe the issue you encountered:
In chrome and FF if you drag and drop an HTML file from a folder to the Browser header section (where the tabs are located) they correctly open the local file in a new tab so you can preview it.
In Brave this is currently not possible.

Expected behavior:
Brave should open local html files for preview via drag and drop.

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

• Brave Version:
  0.12.0

• Steps to reproduce:

  1. Open a folder containing html files
  2. Drag one of the to the Brave's header section
  3. Nothing happens

• Screenshot if needed:

• Any related issues:

Test Plan:

1. Open folder with a local file
2. Drag and drop one to the header section
3. File is opened

[bsclifton]

Fixed with #6404

[luixxiul]

The file path is instead searched:

[NejcZdovc]

I see where the problem is. If you dnd on a tab section it will be loaded correctly, but if you dnd it into URL bar, path will be searched. This of course happens only when you add link that doesn have space in path for example. Will fix it.

[bsclifton]

PR addressing this issue was merged after I confirmed the fix; It will be in the next RC 👍

[luixxiul]

if you dnd it into URL bar

Sorry for the bad gif demonstration, but the error can happen as well dragging the file onto the tab:

[luixxiul]

Reopened just in case; please close this if #7057 has fixed it too.

[alexwykoff]

confirmed this is an issue in rc3 on win ia-32

cc: @bsclifton

[NejcZdovc]

@alexwykoff @luixxiul this fix is not in rc3 yet, please try it in the next RC

[luixxiul]

@NejcZdovc I thought you have fixed the drag and drop on the URL bar. I just wanted to make sure you have fixed the drag and drop on the tabs bar as well, which the new gif demonstrates on #3819 (comment)

[NejcZdovc]

Will try all scenarious and see if I can reproduce this with the latest code. I wrote about RC's, because @alexwykoff said that he can reproduce it in rc 3, which does not have the latest code. Can you reproduce this latest bug on mac as well?

[bsclifton]

@luixxiul #7057 fixes the issue for sure; I didn't try the tabs bar. Like @NejcZdovc is saying, this was fixed with the PR (#7057) so it's not in a release candidate yet

@NejcZdovc if it works as expected for you in master, please close out the issue 😄

[bbondy]

I think this works as it was spec'ed originally so I'm going to close this.

For follow up work though I posted this one as a spin off:
#7079