diff options
| author | mmenke <mmenke@chromium.org> | 2016-03-23 15:06:45 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-03-23 22:08:06 +0000 |
| commit | 7447f3680fa750cec9dbe27e062a51b5cf8a7e57 (patch) | |
| tree | 2c5c0ed65dbb6980b6893e88955b2e9980fd927f /content/test | |
| parent | 5a941ea1cd19d6a92d197e16ab4508652ff139d0 (diff) | |
| download | chromium_src-7447f3680fa750cec9dbe27e062a51b5cf8a7e57.zip chromium_src-7447f3680fa750cec9dbe27e062a51b5cf8a7e57.tar.gz chromium_src-7447f3680fa750cec9dbe27e062a51b5cf8a7e57.tar.bz2 | |
Remove some uses of UnescapeRule::URL_SPECIAL_CHARS outside net/.
We're removing this, in favor of PATH_SEPARATORS and
URL_SPECIAL_CHARS_EXCEPT_PATH_SEPARATORS, to reduce the
risk of security issues when unescaping, going forward.
BUG=589257
Review URL: https://codereview.chromium.org/1820333003
Cr-Commit-Position: refs/heads/master@{#382935}
Diffstat (limited to 'content/test')
| -rw-r--r-- | content/test/mock_google_streaming_server.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/content/test/mock_google_streaming_server.cc b/content/test/mock_google_streaming_server.cc index 3484ca3..44b48d0 100644 --- a/content/test/mock_google_streaming_server.cc +++ b/content/test/mock_google_streaming_server.cc @@ -46,9 +46,9 @@ void MockGoogleStreamingServer::OnRequestStart(int fetcher_id) { // Extract request argument from the the request URI. std::string query = GetURLFetcher(true)->GetOriginalURL().query(); const net::UnescapeRule::Type kUnescapeAll = - net::UnescapeRule::NORMAL | - net::UnescapeRule::SPACES | - net::UnescapeRule::URL_SPECIAL_CHARS | + net::UnescapeRule::NORMAL | net::UnescapeRule::SPACES | + net::UnescapeRule::PATH_SEPARATORS | + net::UnescapeRule::URL_SPECIAL_CHARS_EXCEPT_PATH_SEPARATORS | net::UnescapeRule::REPLACE_PLUS_WITH_SPACE; for (const base::StringPiece& query_param : base::SplitStringPiece(query, "&", base::KEEP_WHITESPACE, |