diff options
author | gspencer@google.com <gspencer@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-01 00:00:43 +0000 |
---|---|---|
committer | gspencer@google.com <gspencer@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-01 00:00:43 +0000 |
commit | 7a70650e6723fc83f2cc4b1b007d4be1feebc6f3 (patch) | |
tree | ca0a4976a29e7ab5e5ddc88fe98d88871dab3279 /chrome/browser/browser_url_handler.cc | |
parent | 2b79071534c29fc18317bb5c09ce54293da4fd91 (diff) | |
download | chromium_src-7a70650e6723fc83f2cc4b1b007d4be1feebc6f3.zip chromium_src-7a70650e6723fc83f2cc4b1b007d4be1feebc6f3.tar.gz chromium_src-7a70650e6723fc83f2cc4b1b007d4be1feebc6f3.tar.bz2 |
This fixes an issue where view source on a local file would return a
blank page.
This bug was introduced by the fix for issue 26129 (r30418), which
was to limit view source so that it wouldn't be able to load
javascript or data URLs, but that change forgot to allow file urls.
This fixes the above omission by allowing file URLs again.
TEST=works:view-source:file:///tmp/test.html, doesn't work:view-source:javascript:alert('foo')
BUG=27409
Review URL: http://codereview.chromium.org/434094
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33380 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_url_handler.cc')
-rw-r--r-- | chrome/browser/browser_url_handler.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/browser_url_handler.cc b/chrome/browser/browser_url_handler.cc index a8a96e2..0cd1e2f 100644 --- a/chrome/browser/browser_url_handler.cc +++ b/chrome/browser/browser_url_handler.cc @@ -22,7 +22,7 @@ static bool HandleViewSource(GURL* url, Profile* profile) { // other kind of 'active' url scheme like 'javascript' or 'data'. static const char* const allowed_sub_schemes[] = { chrome::kHttpScheme, chrome::kHttpsScheme, chrome::kFtpScheme, - chrome::kChromeUIScheme + chrome::kChromeUIScheme, chrome::kFileScheme }; bool is_sub_scheme_allowed = false; |