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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc

Issue 2261103002: Use modified URLRequest::Read() and delegate methods in components/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@URLRequestRead
Patch Set: comments Created 4 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
Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc
index 7a3b4b1b25e6e02111ca7d3df70ef5c7db0e3520..e44a9c2143f12741ff1dd992fb535b39d6b1dfe8 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc
@@ -245,8 +245,7 @@ class DataReductionProxyProtocolTest : public testing::Test {
base::RunLoop().Run();
if (!expected_error) {
- EXPECT_EQ(net::URLRequestStatus::SUCCESS, r->status().status());
- EXPECT_EQ(net::OK, r->status().error());
+ EXPECT_EQ(net::OK, d.request_status());
if (expected_retry)
EXPECT_EQ(initial_headers_received_count + 2,
network_delegate_->headers_received_count());
@@ -256,8 +255,7 @@ class DataReductionProxyProtocolTest : public testing::Test {
EXPECT_EQ(content, d.data_received());
return;
}
- EXPECT_EQ(net::URLRequestStatus::FAILED, r->status().status());
- EXPECT_EQ(net::ERR_INVALID_RESPONSE, r->status().error());
+ EXPECT_EQ(net::ERR_INVALID_RESPONSE, d.request_status());
EXPECT_EQ(initial_headers_received_count,
network_delegate_->headers_received_count());
}
@@ -980,8 +978,7 @@ TEST_F(DataReductionProxyProtocolTest,
r->Start();
base::RunLoop().Run();
- EXPECT_EQ(net::URLRequestStatus::SUCCESS, r->status().status());
- EXPECT_EQ(net::OK, r->status().error());
+ EXPECT_EQ(net::OK, d.request_status());
EXPECT_EQ("Bypass message", d.data_received());

Powered by Google App Engine
This is Rietveld 408576698