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

Unified Diff: net/url_request/url_request_http_job.cc

Issue 6749044: Remove async functionality from net::CookiePolicy. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 9 years, 8 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_http_job.h ('k') | net/url_request/url_request_test_util.h » ('j') | 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 b3eaa0c8b39e9a7cd43078d84b040315450cee53..851b0d9185bc867bb952ec60333f178171e06f93 100644
--- a/net/url_request/url_request_http_job.cc
+++ b/net/url_request/url_request_http_job.cc
@@ -206,10 +206,6 @@ URLRequestHttpJob::URLRequestHttpJob(URLRequest* request)
response_cookies_save_index_(0),
proxy_auth_state_(AUTH_STATE_DONT_NEED_AUTH),
server_auth_state_(AUTH_STATE_DONT_NEED_AUTH),
- ALLOW_THIS_IN_INITIALIZER_LIST(can_get_cookies_callback_(
- this, &URLRequestHttpJob::OnCanGetCookiesCompleted)),
- ALLOW_THIS_IN_INITIALIZER_LIST(can_set_cookie_callback_(
- this, &URLRequestHttpJob::OnCanSetCookieCompleted)),
ALLOW_THIS_IN_INITIALIZER_LIST(start_callback_(
this, &URLRequestHttpJob::OnStartCompleted)),
ALLOW_THIS_IN_INITIALIZER_LIST(read_callback_(
@@ -418,8 +414,6 @@ void URLRequestHttpJob::AddCookieHeaderAndStart() {
// be notifying our consumer asynchronously via OnStartCompleted.
SetStatus(URLRequestStatus(URLRequestStatus::IO_PENDING, 0));
- AddRef(); // Balanced in OnCanGetCookiesCompleted
-
int policy = OK;
if (request_info_.load_flags & LOAD_DO_NOT_SEND_COOKIES) {
@@ -427,10 +421,7 @@ void URLRequestHttpJob::AddCookieHeaderAndStart() {
} else if (request_->context()->cookie_policy()) {
policy = request_->context()->cookie_policy()->CanGetCookies(
request_->url(),
- request_->first_party_for_cookies(),
- &can_get_cookies_callback_);
- if (policy == ERR_IO_PENDING)
- return; // Wait for completion callback
+ request_->first_party_for_cookies());
}
OnCanGetCookiesCompleted(policy);
@@ -464,8 +455,6 @@ void URLRequestHttpJob::SaveNextCookie() {
// be notifying our consumer asynchronously via OnStartCompleted.
SetStatus(URLRequestStatus(URLRequestStatus::IO_PENDING, 0));
- AddRef(); // Balanced in OnCanSetCookieCompleted
-
int policy = OK;
if (request_info_.load_flags & LOAD_DO_NOT_SAVE_COOKIES) {
@@ -474,10 +463,7 @@ void URLRequestHttpJob::SaveNextCookie() {
policy = request_->context()->cookie_policy()->CanSetCookie(
request_->url(),
request_->first_party_for_cookies(),
- response_cookies_[response_cookies_save_index_],
- &can_set_cookie_callback_);
- if (policy == ERR_IO_PENDING)
- return; // Wait for completion callback
+ response_cookies_[response_cookies_save_index_]);
}
OnCanSetCookieCompleted(policy);
@@ -610,7 +596,6 @@ void URLRequestHttpJob::OnCanGetCookiesCompleted(int policy) {
NotifyCanceled();
}
}
- Release(); // Balance AddRef taken in AddCookieHeaderAndStart
}
void URLRequestHttpJob::OnCanSetCookieCompleted(int policy) {
@@ -649,7 +634,6 @@ void URLRequestHttpJob::OnCanSetCookieCompleted(int policy) {
NotifyCanceled();
}
}
- Release(); // Balance AddRef taken in SaveNextCookie
}
void URLRequestHttpJob::OnStartCompleted(int result) {
« no previous file with comments | « net/url_request/url_request_http_job.h ('k') | net/url_request/url_request_test_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698