Skip to content
This repository has been archived by the owner. It is now read-only.

View page source for local files #8404

Merged
merged 1 commit into from Apr 23, 2017
Merged

View page source for local files #8404

merged 1 commit into from Apr 23, 2017

Conversation

@romamatusevich
Copy link
Contributor

romamatusevich commented Apr 19, 2017

The file uri scheme has been made allowed for view source

fix #8049

Original issue: #8049

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

Test Plan:

  • Open a local HTML file with "File" > "Open File"
  • Right-click anywhere and select "View Page Source"
  • Make sure the source is opened in another tab
@cndouglas
Copy link

cndouglas commented Apr 21, 2017

Thanks @romamatusevich! Unfortunately, there are conflicts because of some recent commits. Could you please resolve them?

@romamatusevich
Copy link
Contributor Author

romamatusevich commented Apr 22, 2017

Hello @Liunkae,

The conflicts were resolved.

Copy link

cndouglas left a comment

Works great!

@cndouglas
Copy link

cndouglas commented Apr 22, 2017

@romamatusevich Please squash the two commits. Thanks for adding tests, by the way!

The file uri scheme has been made allowed for view source

fix #8049
@romamatusevich
Copy link
Contributor Author

romamatusevich commented Apr 22, 2017

@Liunkae, rebased the changes to have one commit only.

@bsclifton
Copy link
Member

bsclifton commented Apr 23, 2017

Awesome!! 👍

Thanks for the patch @romamatusevich 😄 (and thanks @Liunkae for reviewing!)

@bsclifton bsclifton merged commit 135c94f into brave:master Apr 23, 2017
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.