Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(966)

Unified Diff: runtime/vm/uri.cc

Issue 2035123002: Allow double-slash in uris. This is needed for data uri parsing. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | runtime/vm/uri_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/uri.cc
diff --git a/runtime/vm/uri.cc b/runtime/vm/uri.cc
index d23df1573be208c9d7d7e2355b32b8a9d19a0ff9..7d9ef76249bd9902b19890c2b94399fddf5732da 100644
--- a/runtime/vm/uri.cc
+++ b/runtime/vm/uri.cc
@@ -248,20 +248,6 @@ bool ParseUri(const char* uri, ParsedUri* parsed_uri) {
parsed_uri->port = NULL;
}
- // Double slashes in the path do not parse.
- bool saw_slash = false;
- for (const char* pos = path_start; pos < question_pos; pos++) {
- if (*pos == '/') {
- if (saw_slash) {
- ClearParsedUri(parsed_uri);
- return false;
- }
- saw_slash = true;
- } else {
- saw_slash = false;
- }
- }
-
// The path is the substring between the authority and the query.
parsed_uri->path = NormalizeEscapes(path_start, (question_pos - path_start));
return true;
« no previous file with comments | « no previous file | runtime/vm/uri_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698