123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653 |
- // Copyright (c) 2011 The Chromium Authors. All rights reserved.
- // Use of this source code is governed by a BSD-style license that can be
- // found in the LICENSE file.
- #include "components/url_formatter/url_fixer.h"
- #include <stddef.h>
- #include <stdlib.h>
- #include <string>
- #include "base/base_paths.h"
- #include "base/files/file_path.h"
- #include "base/files/file_util.h"
- #include "base/files/scoped_temp_dir.h"
- #include "base/strings/string_util.h"
- #include "base/strings/utf_string_conversions.h"
- #include "build/build_config.h"
- #include "net/base/filename_util.h"
- #include "testing/gtest/include/gtest/gtest.h"
- #include "url/gurl.h"
- #include "url/third_party/mozilla/url_parse.h"
- struct SegmentCase {
- const std::string input;
- const std::string result;
- const url::Component scheme;
- const url::Component username;
- const url::Component password;
- const url::Component host;
- const url::Component port;
- const url::Component path;
- const url::Component query;
- const url::Component ref;
- };
- static const SegmentCase segment_cases[] = {
- {
- "http://www.google.com/", "http", url::Component(0, 4), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(7, 14), // host
- url::Component(), // port
- url::Component(21, 1), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- "aBoUt:vErSiOn", "about", url::Component(0, 5), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(6, 7), // host
- url::Component(), // port
- url::Component(), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- "about:host/path?query#ref", "about", url::Component(0, 5), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(6, 4), // host
- url::Component(), // port
- url::Component(10, 5), // path
- url::Component(16, 5), // query
- url::Component(22, 3), // ref
- },
- {
- "about://host/path?query#ref", "about", url::Component(0, 5), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(8, 4), // host
- url::Component(), // port
- url::Component(12, 5), // path
- url::Component(18, 5), // query
- url::Component(24, 3), // ref
- },
- {
- "chrome:host/path?query#ref", "chrome", url::Component(0, 6), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(7, 4), // host
- url::Component(), // port
- url::Component(11, 5), // path
- url::Component(17, 5), // query
- url::Component(23, 3), // ref
- },
- {
- "chrome://host/path?query#ref", "chrome",
- url::Component(0, 6), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(9, 4), // host
- url::Component(), // port
- url::Component(13, 5), // path
- url::Component(19, 5), // query
- url::Component(25, 3), // ref
- },
- {
- " www.google.com:124?foo#", "http",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(4, 14), // host
- url::Component(19, 3), // port
- url::Component(), // path
- url::Component(23, 3), // query
- url::Component(27, 0), // ref
- },
- {
- "user@www.google.com", "http",
- url::Component(), // scheme
- url::Component(0, 4), // username
- url::Component(), // password
- url::Component(5, 14), // host
- url::Component(), // port
- url::Component(), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- "ftp:/user:P:a$$Wd@..ftp.google.com...::23///pub?foo#bar", "ftp",
- url::Component(0, 3), // scheme
- url::Component(5, 4), // username
- url::Component(10, 7), // password
- url::Component(18, 20), // host
- url::Component(39, 2), // port
- url::Component(41, 6), // path
- url::Component(48, 3), // query
- url::Component(52, 3), // ref
- },
- {
- "[2001:db8::1]/path", "http",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(0, 13), // host
- url::Component(), // port
- url::Component(13, 5), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- "[::1]", "http",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(0, 5), // host
- url::Component(), // port
- url::Component(), // path
- url::Component(), // query
- url::Component(), // ref
- },
- // Incomplete IPv6 addresses (will not canonicalize).
- {
- "[2001:4860:", "http",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(0, 11), // host
- url::Component(), // port
- url::Component(), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- "[2001:4860:/foo", "http",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(0, 11), // host
- url::Component(), // port
- url::Component(11, 4), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- "http://:b005::68]", "http", url::Component(0, 4), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(7, 10), // host
- url::Component(), // port
- url::Component(), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- ":b005::68]", "http",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(1, 9), // host
- url::Component(), // port
- url::Component(), // path
- url::Component(), // query
- url::Component(), // ref
- },
- {
- "file://host/path/file#ref", "file", url::Component(0, 4), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(7, 4), // host
- url::Component(), // port
- url::Component(11, 10), // path
- url::Component(), // query
- url::Component(22, 3), // ref
- },
- {
- "file:///notahost/path/file#ref", "file",
- url::Component(0, 4), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(), // host
- url::Component(), // port
- url::Component(7, 19), // path
- url::Component(), // query
- url::Component(27, 3), // ref
- },
- #if BUILDFLAG(IS_WIN)
- {
- "c:/notahost/path/file#ref", "file",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(), // host
- url::Component(), // port
- url::Component(0, 21), // path
- url::Component(), // query
- url::Component(22, 3), // ref
- },
- #elif BUILDFLAG(IS_POSIX)
- {
- "~/notahost/path/file#ref", "file",
- url::Component(), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(), // host
- url::Component(), // port
- url::Component(0, 20), // path
- url::Component(), // query
- url::Component(21, 3), // ref
- },
- #endif
- {
- "devtools://bundled/devtools/inspector.html?ws=localhost:9221",
- "devtools", url::Component(0, 8), // scheme
- url::Component(), // username
- url::Component(), // password
- url::Component(11, 7), // host
- url::Component(), // port
- url::Component(18, 24), // path
- url::Component(43, 17), // query
- url::Component(), // ref
- },
- };
- typedef testing::Test URLFixerTest;
- TEST(URLFixerTest, SegmentURL) {
- std::string result;
- url::Parsed parts;
- for (size_t i = 0; i < std::size(segment_cases); ++i) {
- SegmentCase value = segment_cases[i];
- SCOPED_TRACE(testing::Message() << "test #" << i << ": " << value.input);
- result = url_formatter::SegmentURL(value.input, &parts);
- EXPECT_EQ(value.result, result);
- EXPECT_EQ(value.scheme, parts.scheme);
- EXPECT_EQ(value.username, parts.username);
- EXPECT_EQ(value.password, parts.password);
- EXPECT_EQ(value.host, parts.host);
- EXPECT_EQ(value.port, parts.port);
- EXPECT_EQ(value.path, parts.path);
- EXPECT_EQ(value.query, parts.query);
- EXPECT_EQ(value.ref, parts.ref);
- }
- }
- // Creates a file and returns its full name as well as the decomposed
- // version. Example:
- // full_path = "c:\foo\bar.txt"
- // dir = "c:\foo"
- // file_name = "bar.txt"
- static bool MakeTempFile(const base::FilePath& dir,
- const base::FilePath& file_name,
- base::FilePath* full_path) {
- *full_path = dir.Append(file_name);
- return base::WriteFile(*full_path, "", 0) == 0;
- }
- // Returns true if the given URL is a file: URL that matches the given file
- static bool IsMatchingFileURL(const std::string& url,
- const base::FilePath& full_file_path) {
- if (url.length() <= 8)
- return false;
- if (std::string("file:///") != url.substr(0, 8))
- return false; // no file:/// prefix
- if (url.find('\\') != std::string::npos)
- return false; // contains backslashes
- base::FilePath derived_path;
- net::FileURLToFilePath(GURL(url), &derived_path);
- return base::FilePath::CompareEqualIgnoreCase(derived_path.value(),
- full_file_path.value());
- }
- struct FixupCase {
- const std::string input;
- const std::string output;
- } fixup_cases[] = {
- {"www.google.com", "http://www.google.com/"},
- {" www.google.com ", "http://www.google.com/"},
- {" foo.com/asdf bar", "http://foo.com/asdf%20%20bar"},
- {"..www.google.com..", "http://www.google.com./"},
- {"http://......", "http://....../"},
- {"http://host.com:ninety-two/", "http://host.com:ninety-two/"},
- {"http://host.com:ninety-two?foo", "http://host.com:ninety-two/?foo"},
- {"google.com:123", "http://google.com:123/"},
- {"about:", "chrome://version/"},
- {"about:foo", "chrome://foo/"},
- {"about:version", "chrome://version/"},
- {"about:blank", "about:blank"},
- {"About:blaNk", "about:blank"},
- {"about:blank#blah", "about:blank#blah"},
- {"about:blank/#blah", "about:blank/#blah"},
- {"about:srcdoc", "about:srcdoc"},
- {"about:srcdoc#blah", "about:srcdoc#blah"},
- {"about:srcdoc/#blah", "about:srcdoc/#blah"},
- {"about:usr:pwd@hst:20/pth?qry#ref", "chrome://hst/pth?qry#ref"},
- {"about://usr:pwd@hst/pth?qry#ref", "chrome://hst/pth?qry#ref"},
- {"chrome:usr:pwd@hst/pth?qry#ref", "chrome://hst/pth?qry#ref"},
- {"chrome://usr:pwd@hst/pth?qry#ref", "chrome://hst/pth?qry#ref"},
- {"www:123", "http://www:123/"},
- {" www:123", "http://www:123/"},
- {"www.google.com?foo", "http://www.google.com/?foo"},
- {"www.google.com#foo", "http://www.google.com/#foo"},
- {"www.google.com?", "http://www.google.com/?"},
- {"www.google.com#", "http://www.google.com/#"},
- {"www.google.com:123?foo#bar", "http://www.google.com:123/?foo#bar"},
- {"user@www.google.com", "http://user@www.google.com/"},
- {"\xE6\xB0\xB4.com", "http://xn--1rw.com/"},
- // It would be better if this next case got treated as http, but I don't see
- // a clean way to guess this isn't the new-and-exciting "user" scheme.
- {"user:passwd@www.google.com:8080/", "user:passwd@www.google.com:8080/"},
- // {"file:///c:/foo/bar%20baz.txt", "file:///C:/foo/bar%20baz.txt"},
- // URLs which end with 0x85 (NEL in ISO-8859).
- {"http://example.com/s?q=\xD0\x85", "http://example.com/s?q=%D0%85"},
- {"http://example.com/s?q=\xEC\x97\x85", "http://example.com/s?q=%EC%97%85"},
- {"http://example.com/s?q=\xF0\x90\x80\x85",
- "http://example.com/s?q=%F0%90%80%85"},
- // URLs which end with 0xA0 (non-break space in ISO-8859).
- {"http://example.com/s?q=\xD0\xA0", "http://example.com/s?q=%D0%A0"},
- {"http://example.com/s?q=\xEC\x97\xA0", "http://example.com/s?q=%EC%97%A0"},
- {"http://example.com/s?q=\xF0\x90\x80\xA0",
- "http://example.com/s?q=%F0%90%80%A0"},
- // URLs containing Unicode non-characters.
- {"http://example.com/s?q=\xEF\xB7\x90", // U+FDD0
- "http://example.com/s?q=%EF%BF%BD"},
- {"http://example.com/s?q=\xEF\xBF\xBE", // U+FFFE
- "http://example.com/s?q=%EF%BF%BD"},
- {"http://example.com/s?q=\xEF\xBF\xBF", // U+FFFF
- "http://example.com/s?q=%EF%BF%BD"},
- {"http://example.com/s?q=\xF4\x8F\xBF\xBE", // U+10FFFE
- "http://example.com/s?q=%EF%BF%BD"},
- {"http://example.com/s?q=\xF4\x8F\xBF\xBF", // U+10FFFF
- "http://example.com/s?q=%EF%BF%BD"},
- // URLs containing IPv6 literals.
- {"[2001:db8::2]", "http://[2001:db8::2]/"},
- {"[::]:80", "http://[::]/"},
- {"[::]:80/path", "http://[::]/path"},
- {"[::]:180/path", "http://[::]:180/path"},
- // TODO(pmarks): Maybe we should parse bare IPv6 literals someday. Currently
- // the first colon is treated as a scheme separator, and we default
- // unspecified schemes to "http".
- {"::1", "http://:1/"},
- // Semicolon as scheme separator for standard schemes.
- {"http;//www.google.com/", "http://www.google.com/"},
- {"about;help", "chrome://help/"},
- // Semicolon in non-standard schemes is not replaced by colon.
- {"whatsup;//fool", "http://whatsup%3B//fool"},
- // Semicolon left as-is in URL itself.
- {"http://host/port?query;moar", "http://host/port?query;moar"},
- // Fewer slashes than expected.
- {"http;www.google.com/", "http://www.google.com/"},
- {"http;/www.google.com/", "http://www.google.com/"},
- // Semicolon at start.
- {";http://www.google.com/", "http://%3Bhttp//www.google.com/"},
- // DevTools scheme.
- {"devtools://bundled/devtools/node.html",
- "devtools://bundled/devtools/node.html"},
- // DevTools scheme with websocket query.
- {"devtools://bundled/devtools/inspector.html?ws=ws://localhost:9222/guid",
- "devtools://bundled/devtools/inspector.html?ws=ws://localhost:9222/guid"},
- // host:123 should be rewritten to http://host:123/, but only if the port
- // number is valid - in particular telephone numbers are not port numbers
- // (see also SendTabToSelfUtilTest.ShouldNotOfferFeatureForTelephoneLink).
- {"host:123", "http://host:123/"},
- {"host:80", "http://host/"}, // default port is removed
- {"host:9999", "http://host:9999/"},
- {"host:00009999", "http://host:9999/"}, // leading zeros are removed
- {"host:0", "http://host:0/"}, // min valid port
- {"host:65535", "http://host:65535/"}, // max valid port
- {"host:-1", "host:-1"},
- {"host:65536", "host:65536"},
- {"host:18446744073709551619", "host:18446744073709551619"}, // > uint64.max
- {"host:", "host:"},
- {"host: 123", "host: 123"},
- {"host:+123", "host:+123"},
- {"host:1.23", "host:1.23"},
- {"host:x", "host:x"},
- {"host:᠐", "host:%E1%A0%90"}, // non-ASCII digit (U+1810)
- {"host:𝟨", "host:%F0%9D%9F%A8"}, // non-ASCII digit (U+1D7E8)
- {"tel:12345678901", "tel:12345678901"},
- {"tel:123-456-78901", "tel:123-456-78901"},
- // Double colon after host should not convert to an empty port.
- {"foo.com::/server-redirect?http%3A%2F%2Fbar.com%2Ftitle2.html",
- "http://foo.com/server-redirect?http%3A%2F%2Fbar.com%2Ftitle2.html"},
- };
- TEST(URLFixerTest, FixupURL) {
- for (const auto& value : fixup_cases) {
- GURL actual_output = url_formatter::FixupURL(value.input, std::string());
- EXPECT_EQ(value.output, actual_output.possibly_invalid_spec())
- << "input: " << value.input;
- // Fixup URL should never translate a valid GURL into an invalid one.
- if (GURL(value.input).is_valid())
- EXPECT_TRUE(actual_output.is_valid());
- }
- // Check the TLD-appending functionality.
- FixupCase tld_cases[] = {
- {"somedomainthatwillnotbeagtld",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"somedomainthatwillnotbeagtld.",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"somedomainthatwillnotbeagtld..",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {".somedomainthatwillnotbeagtld",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"www.somedomainthatwillnotbeagtld",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"somedomainthatwillnotbeagtld.com",
- "http://somedomainthatwillnotbeagtld.com/"},
- {"http://somedomainthatwillnotbeagtld",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"..somedomainthatwillnotbeagtld..",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"http://www.somedomainthatwillnotbeagtld",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"9999999999999999", "http://www.9999999999999999.com/"},
- {"somedomainthatwillnotbeagtld/foo",
- "http://www.somedomainthatwillnotbeagtld.com/foo"},
- {"somedomainthatwillnotbeagtld.com/foo",
- "http://somedomainthatwillnotbeagtld.com/foo"},
- {"somedomainthatwillnotbeagtld/?foo=.com",
- "http://www.somedomainthatwillnotbeagtld.com/?foo=.com"},
- {"www.somedomainthatwillnotbeagtld/?foo=www.",
- "http://www.somedomainthatwillnotbeagtld.com/?foo=www."},
- {"somedomainthatwillnotbeagtld.com/?foo=.com",
- "http://somedomainthatwillnotbeagtld.com/?foo=.com"},
- {"http://www.somedomainthatwillnotbeagtld.com",
- "http://www.somedomainthatwillnotbeagtld.com/"},
- {"somedomainthatwillnotbeagtld:123",
- "http://www.somedomainthatwillnotbeagtld.com:123/"},
- {"http://somedomainthatwillnotbeagtld:123",
- "http://www.somedomainthatwillnotbeagtld.com:123/"},
- };
- for (const auto& value : tld_cases) {
- EXPECT_EQ(
- value.output,
- url_formatter::FixupURL(value.input, "com").possibly_invalid_spec());
- }
- }
- // Test different types of file inputs to URIFixerUpper::FixupURL. This
- // doesn't go into the nice array of fixups above since the file input
- // has to exist.
- TEST(URLFixerTest, FixupFile) {
- // this "original" filename is the one we tweak to get all the variations
- base::ScopedTempDir temp_dir_;
- ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
- base::FilePath original;
- ASSERT_TRUE(MakeTempFile(
- temp_dir_.GetPath(),
- base::FilePath(FILE_PATH_LITERAL("url fixer upper existing file.txt")),
- &original));
- // reference path
- GURL golden(net::FilePathToFileURL(original));
- // c:\foo\bar.txt -> file:///c:/foo/bar.txt (basic)
- GURL fixedup(url_formatter::FixupURL(original.AsUTF8Unsafe(), std::string()));
- EXPECT_EQ(golden, fixedup);
- // TODO(port): Make some equivalent tests for posix.
- #if BUILDFLAG(IS_WIN)
- // c|/foo\bar.txt -> file:///c:/foo/bar.txt (pipe allowed instead of colon)
- std::string cur(base::WideToUTF8(original.value()));
- EXPECT_EQ(':', cur[1]);
- cur[1] = '|';
- EXPECT_EQ(golden, url_formatter::FixupURL(cur, std::string()));
- FixupCase cases[] = {
- {"c:\\Non-existent%20file.txt", "file:///C:/Non-existent%2520file.txt"},
- // \\foo\bar.txt -> file://foo/bar.txt
- // UNC paths, this file won't exist, but since there are no escapes, it
- // should be returned just converted to a file: URL.
- {"\\\\NonexistentHost\\foo\\bar.txt", "file://nonexistenthost/foo/bar.txt"},
- // We do this strictly, like IE8, which only accepts this form using
- // backslashes and not forward ones. Turning "//foo" into "http" matches
- // Firefox and IE, silly though it may seem (it falls out of adding "http"
- // as the default protocol if you haven't entered one).
- {"//NonexistentHost\\foo/bar.txt", "http://nonexistenthost/foo/bar.txt"},
- {"file:///C:/foo/bar", "file:///C:/foo/bar"},
- // Much of the work here comes from GURL's canonicalization stage.
- {"file://C:/foo/bar", "file:///C:/foo/bar"},
- {"file:c:", "file:///C:"},
- {"file:c:WINDOWS", "file:///C:/WINDOWS"},
- {"file:c|Program Files", "file:///C:/Program%20Files"},
- {"file:/file", "file://file/"},
- {"file:////////c:\\foo", "file:///C:/foo"},
- {"file://server/folder/file", "file://server/folder/file"},
- // These are fixups we don't do, but could consider:
- // {"file:///foo:/bar", "file://foo/bar"},
- // {"file:/\\/server\\folder/file", "file://server/folder/file"},
- };
- #elif BUILDFLAG(IS_POSIX) || BUILDFLAG(IS_FUCHSIA)
- #if BUILDFLAG(IS_APPLE)
- #define HOME "/Users/"
- #else
- #define HOME "/home/"
- #endif
- url_formatter::home_directory_override = "/foo";
- FixupCase cases[] = {
- // File URLs go through GURL, which tries to escape intelligently.
- {"/A%20non-existent file.txt", "file:///A%2520non-existent%20file.txt"},
- // A plain "/" refers to the root.
- {"/", "file:///"},
- // These rely on the above home_directory_override.
- {"~", "file:///foo"},
- {"~/bar", "file:///foo/bar"},
- // References to other users' homedirs.
- {"~foo", "file://" HOME "foo"},
- {"~x/blah", "file://" HOME "x/blah"},
- };
- #endif
- for (const auto& value : cases) {
- EXPECT_EQ(value.output, url_formatter::FixupURL(value.input, std::string())
- .possibly_invalid_spec());
- }
- EXPECT_TRUE(base::DeleteFile(original));
- }
- TEST(URLFixerTest, FixupRelativeFile) {
- base::FilePath full_path;
- base::FilePath file_part(
- FILE_PATH_LITERAL("url_fixer_upper_existing_file.txt"));
- base::ScopedTempDir temp_dir_;
- ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
- ASSERT_TRUE(MakeTempFile(temp_dir_.GetPath(), file_part, &full_path));
- full_path = base::MakeAbsoluteFilePath(full_path);
- ASSERT_FALSE(full_path.empty());
- // make sure we pass through good URLs
- for (const auto& value : fixup_cases) {
- base::FilePath input = base::FilePath::FromUTF8Unsafe(value.input);
- EXPECT_EQ(value.output,
- url_formatter::FixupRelativeFile(temp_dir_.GetPath(), input)
- .possibly_invalid_spec())
- << "input: " << value.input;
- }
- // make sure the existing file got fixed-up to a file URL, and that there
- // are no backslashes
- EXPECT_TRUE(IsMatchingFileURL(
- url_formatter::FixupRelativeFile(temp_dir_.GetPath(), file_part)
- .possibly_invalid_spec(),
- full_path));
- EXPECT_TRUE(base::DeleteFile(full_path));
- // create a filename we know doesn't exist and make sure it doesn't get
- // fixed up to a file URL
- base::FilePath nonexistent_file(
- FILE_PATH_LITERAL("url_fixer_upper_nonexistent_file.txt"));
- std::string fixedup(
- url_formatter::FixupRelativeFile(temp_dir_.GetPath(), nonexistent_file)
- .possibly_invalid_spec());
- EXPECT_NE(std::string("file:///"), fixedup.substr(0, 8));
- EXPECT_FALSE(IsMatchingFileURL(fixedup, nonexistent_file));
- // make a subdir to make sure relative paths with directories work, also
- // test spaces:
- // "app_dir\url fixer-upper dir\url fixer-upper existing file.txt"
- base::FilePath sub_dir(FILE_PATH_LITERAL("url fixer-upper dir"));
- base::FilePath sub_file(
- FILE_PATH_LITERAL("url fixer-upper existing file.txt"));
- base::FilePath new_dir = temp_dir_.GetPath().Append(sub_dir);
- base::CreateDirectory(new_dir);
- ASSERT_TRUE(MakeTempFile(new_dir, sub_file, &full_path));
- full_path = base::MakeAbsoluteFilePath(full_path);
- ASSERT_FALSE(full_path.empty());
- // test file in the subdir
- base::FilePath relative_file = sub_dir.Append(sub_file);
- EXPECT_TRUE(IsMatchingFileURL(
- url_formatter::FixupRelativeFile(temp_dir_.GetPath(), relative_file)
- .possibly_invalid_spec(),
- full_path));
- // test file in the subdir with different slashes and escaping.
- base::FilePath::StringType relative_file_str = sub_dir.value() +
- FILE_PATH_LITERAL("/") + sub_file.value();
- base::ReplaceSubstringsAfterOffset(&relative_file_str, 0,
- FILE_PATH_LITERAL(" "), FILE_PATH_LITERAL("%20"));
- EXPECT_TRUE(IsMatchingFileURL(
- url_formatter::FixupRelativeFile(temp_dir_.GetPath(),
- base::FilePath(relative_file_str))
- .possibly_invalid_spec(),
- full_path));
- // test relative directories and duplicate slashes
- // (should resolve to the same file as above)
- relative_file_str = sub_dir.value() + FILE_PATH_LITERAL("/../") +
- sub_dir.value() + FILE_PATH_LITERAL("///./") + sub_file.value();
- EXPECT_TRUE(IsMatchingFileURL(
- url_formatter::FixupRelativeFile(temp_dir_.GetPath(),
- base::FilePath(relative_file_str))
- .possibly_invalid_spec(),
- full_path));
- // done with the subdir
- EXPECT_TRUE(base::DeleteFile(full_path));
- EXPECT_TRUE(base::DeletePathRecursively(new_dir));
- // Test that an obvious HTTP URL isn't accidentally treated as an absolute
- // file path (on account of system-specific craziness).
- base::FilePath empty_path;
- base::FilePath http_url_path(FILE_PATH_LITERAL("http://../"));
- EXPECT_TRUE(url_formatter::FixupRelativeFile(empty_path, http_url_path)
- .SchemeIs("http"));
- }
|