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

Unified Diff: net/url_request/url_request_http_job.cc

Issue 9700056: Remove experimental support for MAC cookies (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove unused global Created 8 years, 9 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 | « net/url_request/url_request.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/url_request_http_job.cc
diff --git a/net/url_request/url_request_http_job.cc b/net/url_request/url_request_http_job.cc
index d52c35132add2764333c90cbcc8e9a835c71a726..69993bd9eb9cf1dcf3265abb7ae0da477d2314d1 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -29,7 +29,6 @@
#include "net/base/sdch_manager.h"
#include "net/base/ssl_cert_request_info.h"
#include "net/base/ssl_config_service.h"
-#include "net/http/http_mac_signature.h"
#include "net/http/http_request_headers.h"
#include "net/http/http_response_headers.h"
#include "net/http/http_response_info.h"
@@ -49,37 +48,6 @@ static const char kAvailDictionaryHeader[] = "Avail-Dictionary";
namespace net {
-namespace {
-
-void AddAuthorizationHeader(
- const std::vector<CookieStore::CookieInfo>& cookie_infos,
- HttpRequestInfo* request_info) {
- const GURL& url = request_info->url;
- const std::string& method = request_info->method;
- std::string request_uri = HttpUtil::PathForRequest(url);
- const std::string& host = url.host();
- int port = url.EffectiveIntPort();
- for (size_t i = 0; i < cookie_infos.size(); ++i) {
- HttpMacSignature signature;
- if (!signature.AddStateInfo(cookie_infos[i].name,
- cookie_infos[i].creation_date,
- cookie_infos[i].mac_key,
- cookie_infos[i].mac_algorithm)) {
- continue;
- }
- if (!signature.AddHttpInfo(method, request_uri, host, port))
- continue;
- std::string authorization_header;
- if (!signature.GenerateAuthorizationHeader(&authorization_header))
- continue;
- request_info->extra_headers.SetHeader(HttpRequestHeaders::kAuthorization,
- authorization_header);
- return; // Only add the first valid header.
- }
-}
-
-} // namespace
-
class URLRequestHttpJob::HttpFilterContext : public FilterContext {
public:
explicit HttpFilterContext(URLRequestHttpJob* job);
@@ -509,8 +477,6 @@ void URLRequestHttpJob::OnCookiesLoaded(
request_info_.extra_headers.SetHeader(
HttpRequestHeaders::kCookie, cookie_line);
}
- if (URLRequest::AreMacCookiesEnabled())
- AddAuthorizationHeader(cookie_infos, &request_info_);
DoStartTransaction();
}
« no previous file with comments | « net/url_request/url_request.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698