|
|
Reject some previuosly-escaped chars in hostnames.
Previuosly some technically invalid characters were accepted in host names and
escaped when they were encountered. This was done for IE6 compat. However,
these characters now fail on most users' systems so we can remove the
complexity associated with these characters.
These invalid characters are now rejected and any URLs with them in the host
name are marked invalid.
BUG= 652808
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+52 lines, -104 lines) |
Patch |
 |
M |
chrome/test/data/extensions/api_test/cookies/api/manifest.json
|
View
|
1
2
3
4
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
chrome/test/data/extensions/api_test/cookies/api/tab.js
|
View
|
1
2
3
4
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/omnibox/browser/autocomplete_input_unittest.cc
|
View
|
1
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/omnibox/browser/history_url_provider_unittest.cc
|
View
|
1
|
2 chunks |
+4 lines, -10 lines |
0 comments
|
Download
|
|
M |
components/safe_browsing_db/v4_protocol_manager_util.cc
|
View
|
1
2
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
components/safe_browsing_db/v4_protocol_manager_util_unittest.cc
|
View
|
1
2
|
1 chunk |
+2 lines, -4 lines |
0 comments
|
Download
|
|
M |
components/search_engines/template_url_unittest.cc
|
View
|
1
2
|
2 chunks |
+8 lines, -8 lines |
0 comments
|
Download
|
|
M |
content/common/database_identifier_unittest.cc
|
View
|
1
2
3
4
|
5 chunks |
+18 lines, -20 lines |
0 comments
|
Download
|
|
M |
net/base/filename_util_unittest.cc
|
View
|
1
2
3
4
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
net/cert/x509_certificate_unittest.cc
|
View
|
1
2
3
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
net/http/http_security_headers_unittest.cc
|
View
|
1
2
3
|
1 chunk |
+0 lines, -11 lines |
0 comments
|
Download
|
|
M |
net/proxy/proxy_config_service_linux_unittest.cc
|
View
|
1
2
3
|
1 chunk |
+0 lines, -15 lines |
0 comments
|
Download
|
|
M |
url/url_canon_host.cc
|
View
|
1
2
|
3 chunks |
+7 lines, -21 lines |
0 comments
|
Download
|
|
M |
url/url_canon_unittest.cc
|
View
|
1
2
|
4 chunks |
+7 lines, -8 lines |
0 comments
|
Download
|
Total messages: 17 (12 generated)
|
Chromium Code Reviews
Issue
2397873002:
Reject some previuosly-escaped chars in hostnames.
