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

Unified Diff: runtime/bin/http_impl.dart

Issue 11231070: Fix bug in HTTP client request generation (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 2 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/bin/http_parser.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/bin/http_impl.dart
diff --git a/runtime/bin/http_impl.dart b/runtime/bin/http_impl.dart
index 503067c98df362c1c0f2d4ceb928285f6a7a7437..9b9ee048054023c96fefd70dcf8a2d20f97e014d 100644
--- a/runtime/bin/http_impl.dart
+++ b/runtime/bin/http_impl.dart
@@ -1675,15 +1675,14 @@ class _HttpClientRequest
// Send the path for direct connections and the whole URL for
// proxy connections.
if (!_connection._usingProxy) {
- String path;
+ String path = _uri.path;
+ if (path.length == 0) path = "/";
if (_uri.query != "") {
if (_uri.fragment != "") {
- path = "${_uri.path}?${_uri.query}#${_uri.fragment}";
+ path = "${path}?${_uri.query}#${_uri.fragment}";
} else {
- path = "${_uri.path}?${_uri.query}";
+ path = "${path}?${_uri.query}";
}
- } else {
- path = _uri.path;
}
data = path.charCodes();
} else {
« no previous file with comments | « no previous file | runtime/bin/http_parser.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698