Index: chrome/browser/chromeos/fileapi/external_file_url_request_job_unittest.cc |
diff --git a/chrome/browser/chromeos/fileapi/external_file_url_request_job_unittest.cc b/chrome/browser/chromeos/fileapi/external_file_url_request_job_unittest.cc |
index a26137f62101e7c3e4b703f8eb1c3c363e249e41..f8927a9978454137275594429575326d4392d2f9 100644 |
--- a/chrome/browser/chromeos/fileapi/external_file_url_request_job_unittest.cc |
+++ b/chrome/browser/chromeos/fileapi/external_file_url_request_job_unittest.cc |
@@ -108,8 +108,8 @@ class TestDelegate : public net::TestDelegate { |
const net::RedirectInfo& redirect_info, |
bool* defer_redirect) override { |
redirect_url_ = redirect_info.new_url; |
- net::TestDelegate::OnReceivedRedirect( |
- request, redirect_info, defer_redirect); |
+ net::TestDelegate::OnReceivedRedirect(request, redirect_info, |
+ defer_redirect); |
} |
private: |
@@ -249,8 +249,7 @@ TEST_F(ExternalFileURLRequestJobTest, NonGetMethod) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::FAILED, request->status().status()); |
- EXPECT_EQ(net::ERR_METHOD_NOT_SUPPORTED, request->status().error()); |
+ EXPECT_EQ(net::ERR_METHOD_NOT_SUPPORTED, test_dalegate_.request_status()); |
} |
TEST_F(ExternalFileURLRequestJobTest, RegularFile) { |
@@ -266,7 +265,7 @@ TEST_F(ExternalFileURLRequestJobTest, RegularFile) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::SUCCESS, request->status().status()); |
+ EXPECT_EQ(net::OK, test_dalegate_.request_status()); |
// It looks weird, but the mime type for the "File 1.txt" is "audio/mpeg" |
// on the server. |
std::string mime_type; |
@@ -292,7 +291,8 @@ TEST_F(ExternalFileURLRequestJobTest, RegularFile) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::SUCCESS, request->status().status()); |
+ EXPECT_EQ(net::OK, test_dalegate_.request_status()); |
+ |
std::string mime_type; |
request->GetMimeType(&mime_type); |
EXPECT_EQ("audio/mpeg", mime_type); |
@@ -314,7 +314,7 @@ TEST_F(ExternalFileURLRequestJobTest, HostedDocument) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::SUCCESS, request->status().status()); |
+ EXPECT_EQ(net::OK, test_dalegate_.request_status()); |
// Make sure that a hosted document triggers redirection. |
EXPECT_TRUE(request->is_redirecting()); |
EXPECT_TRUE(test_delegate_->redirect_url().is_valid()); |
@@ -328,8 +328,7 @@ TEST_F(ExternalFileURLRequestJobTest, RootDirectory) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::FAILED, request->status().status()); |
- EXPECT_EQ(net::ERR_FAILED, request->status().error()); |
+ EXPECT_EQ(net::ERR_FAILED, test_dalegate_.request_status()); |
} |
TEST_F(ExternalFileURLRequestJobTest, Directory) { |
@@ -340,8 +339,7 @@ TEST_F(ExternalFileURLRequestJobTest, Directory) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::FAILED, request->status().status()); |
- EXPECT_EQ(net::ERR_FAILED, request->status().error()); |
+ EXPECT_EQ(net::ERR_FAILED, test_dalegate_.request_status()); |
} |
TEST_F(ExternalFileURLRequestJobTest, NonExistingFile) { |
@@ -352,8 +350,7 @@ TEST_F(ExternalFileURLRequestJobTest, NonExistingFile) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::FAILED, request->status().status()); |
- EXPECT_EQ(net::ERR_FILE_NOT_FOUND, request->status().error()); |
+ EXPECT_EQ(net::ERR_FILE_NOT_FOUND, test_dalegate_.request_status()); |
} |
TEST_F(ExternalFileURLRequestJobTest, WrongFormat) { |
@@ -363,8 +360,7 @@ TEST_F(ExternalFileURLRequestJobTest, WrongFormat) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::FAILED, request->status().status()); |
- EXPECT_EQ(net::ERR_INVALID_URL, request->status().error()); |
+ EXPECT_EQ(net::ERR_INVALID_URL, test_dalegate_.request_status()); |
} |
TEST_F(ExternalFileURLRequestJobTest, Cancel) { |
@@ -378,7 +374,7 @@ TEST_F(ExternalFileURLRequestJobTest, Cancel) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::CANCELED, request->status().status()); |
+ EXPECT_EQ(net::ERR_ABORTED, test_dalegate_.request_status()); |
} |
TEST_F(ExternalFileURLRequestJobTest, RangeHeader) { |
@@ -395,7 +391,7 @@ TEST_F(ExternalFileURLRequestJobTest, RangeHeader) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::SUCCESS, request->status().status()); |
+ EXPECT_EQ(net::OK, test_dalegate_.request_status()); |
// Reading file must be done after |request| runs, otherwise |
// it'll create a local cache file, and we cannot test correctly. |
@@ -417,8 +413,7 @@ TEST_F(ExternalFileURLRequestJobTest, WrongRangeHeader) { |
base::RunLoop().Run(); |
- EXPECT_EQ(net::URLRequestStatus::FAILED, request->status().status()); |
- EXPECT_EQ(net::ERR_REQUEST_RANGE_NOT_SATISFIABLE, request->status().error()); |
+ EXPECT_EQ(net::ERR_REQUEST_RANGE_NOT_SATISFIABLE, test_delegate_.); |
} |
} // namespace chromeos |