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

Unified Diff: net/http/http_util.cc

Issue 213017: Moving two http utility functions to a separate source file due to dependency... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 3 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 | net/http/http_util_icu.cc » ('j') | net/http/http_util_icu.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_util.cc
===================================================================
--- net/http/http_util.cc (revision 26444)
+++ net/http/http_util.cc (working copy)
@@ -54,20 +54,6 @@
//-----------------------------------------------------------------------------
// static
-std::string HttpUtil::PathForRequest(const GURL& url) {
- DCHECK(url.is_valid() && (url.SchemeIs("http") || url.SchemeIs("https")));
- if (url.has_query())
- return url.path() + "?" + url.query();
- return url.path();
-}
-
-// static
-std::string HttpUtil::SpecForRequest(const GURL& url) {
- DCHECK(url.is_valid() && (url.SchemeIs("http") || url.SchemeIs("https")));
- return SimplifyUrlForRequest(url).spec();
-}
-
-// static
size_t HttpUtil::FindDelimiter(const string& line, size_t search_start,
char delimiter) {
do {
@@ -453,7 +439,7 @@
return i;
}
}
- return -1; // Not found
+ return -1; // Not found
}
int HttpUtil::LocateEndOfHeaders(const char* buf, int buf_len, int i) {
@@ -512,7 +498,7 @@
if (!HttpUtil::IsLWS(*cur))
return cur;
}
- return end; // Not found.
+ return end; // Not found.
}
std::string HttpUtil::AssembleRawHeaders(const char* input_begin,
« no previous file with comments | « no previous file | net/http/http_util_icu.cc » ('j') | net/http/http_util_icu.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698