From 7ec7c1899afc8537bb25ce373aa2e136aa7788dc Mon Sep 17 00:00:00 2001 From: "eroman@chromium.org" Date: Tue, 4 Aug 2009 02:33:58 +0000 Subject: Split out HttpUtil::SpecForRequest() into a more generic function of net_util.h. This was a TODO, since that function is useful outside of HTTP. In the process, I uncovered some test cases in proxy_service that are passing in invalid URLs (by virtue of the extra DCHECK). This doesn't make much sense to me to support that, so I have changed them. Review URL: http://codereview.chromium.org/160558 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22359 0039d316-1c4b-4281-b951-d872f2087c98 --- net/http/http_util.cc | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'net/http') diff --git a/net/http/http_util.cc b/net/http/http_util.cc index 5e81dab..4381117 100644 --- a/net/http/http_util.cc +++ b/net/http/http_util.cc @@ -12,6 +12,7 @@ #include "base/logging.h" #include "base/string_piece.h" #include "base/string_util.h" +#include "net/base/net_util.h" using std::string; @@ -63,11 +64,7 @@ std::string HttpUtil::PathForRequest(const GURL& url) { // static std::string HttpUtil::SpecForRequest(const GURL& url) { DCHECK(url.is_valid() && (url.SchemeIs("http") || url.SchemeIs("https"))); - GURL::Replacements replacements; - replacements.ClearUsername(); - replacements.ClearPassword(); - replacements.ClearRef(); - return url.ReplaceComponents(replacements).spec(); + return SimplifyUrlForRequest(url).spec(); } // static -- cgit v1.1