| commit | ee5e87f4b955407e31ac6357a9b164a9558fcfef | [log] [tgz] |
|---|---|---|
| author | Matt Giuca <[email protected]> | Thu Jun 07 04:18:41 2018 |
| committer | Commit Bot <[email protected]> | Thu Jun 07 04:18:41 2018 |
| tree | 5b0b6bc0b316c469af6cda751c2061c329ae487f | |
| parent | 0c9feff82bdd88452bb151ce650fbe781518c3f9 [diff] |
file URLs now correctly percent-decode when looking up a file on disk.
Since percent-encoded bytes are meaningless in a file system, it is
never correct to keep them percent-encoded, even if they would remain
encoded for display. This fixes file URL mapping onto the file system
for the following examples:
- file:///abc%0Adef.txt (control characters)
- file:///%F0%9F%94%92.txt (blacklisted Unicode characters)
Encoded slashes ("%2F", and "%5C" on Windows) now fail to map onto a
file, rather than remaining encoded. "file:///foo%2Fbar" previously
mapped onto a file named "foo%2Fbar", which makes no sense (such a file
is properly addressed by "file:///foo%252Fbar").
Bug: 585422
Change-Id: Ia8b76276db37c6e987e81f9058a84bc876bfa7df
Reviewed-on: https://chromium-review.googlesource.com/1053672
Commit-Queue: Matt Giuca <[email protected]>
Reviewed-by: Matt Menke <[email protected]>
Cr-Commit-Position: refs/heads/master@{#565176}
ChromiumChromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .