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

Unified Diff: content/common/net/url_fetcher_impl_unittest.cc

Issue 9572001: Do cookie checks in NetworkDelegate instead of the URLRequest::Delegate. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: patch for landing 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 | « content/common/net/url_fetcher_impl.cc ('k') | content/common/net/url_request_user_data.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/net/url_fetcher_impl_unittest.cc
diff --git a/content/common/net/url_fetcher_impl_unittest.cc b/content/common/net/url_fetcher_impl_unittest.cc
index 8b722a410932de8ed80c7953e34ddd3517251bad..b70db14c419b884e3cd391c95be3f31436036afc 100644
--- a/content/common/net/url_fetcher_impl_unittest.cc
+++ b/content/common/net/url_fetcher_impl_unittest.cc
@@ -381,8 +381,9 @@ void URLFetcherProtectTest::OnURLFetchComplete(
static int count = 0;
count++;
if (count < 20) {
- fetcher_->StartWithRequestContextGetter(new TestURLRequestContextGetter(
- io_message_loop_proxy()));
+ fetcher_->SetRequestContext(
+ new TestURLRequestContextGetter(io_message_loop_proxy()));
+ fetcher_->Start();
} else {
// We have already sent 20 requests continuously. And we expect that
// it takes more than 1 second due to the overload protection settings.
@@ -492,8 +493,9 @@ void URLFetcherMultipleAttemptTest::OnURLFetchComplete(
EXPECT_FALSE(data.empty());
if (!data.empty() && data_.empty()) {
data_ = data;
- fetcher_->StartWithRequestContextGetter(
+ fetcher_->SetRequestContext(
new TestURLRequestContextGetter(io_message_loop_proxy()));
+ fetcher_->Start();
} else {
EXPECT_EQ(data, data_);
delete fetcher_; // Have to delete this here and not in the destructor,
« no previous file with comments | « content/common/net/url_fetcher_impl.cc ('k') | content/common/net/url_request_user_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698