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

Unified Diff: components/offline_pages/core/prefetch/prefetch_request_test_base.cc

Issue 2889453003: [Offline Prefetech] Send GetOperationReqest to the server (Closed)
Patch Set: Address feedback Created 3 years, 7 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/offline_pages/core/prefetch/prefetch_request_test_base.cc
diff --git a/components/offline_pages/core/prefetch/prefetch_request_test_base.cc b/components/offline_pages/core/prefetch/prefetch_request_test_base.cc
index d8f8a04252a37042286d956b9351756c7a944f6c..1a6305d14506a7898a456ad058f93b08aa9ac041 100644
--- a/components/offline_pages/core/prefetch/prefetch_request_test_base.cc
+++ b/components/offline_pages/core/prefetch/prefetch_request_test_base.cc
@@ -18,7 +18,7 @@ PrefetchRequestTestBase::PrefetchRequestTestBase()
PrefetchRequestTestBase::~PrefetchRequestTestBase() {}
void PrefetchRequestTestBase::RespondWithNetError(int net_error) {
- net::TestURLFetcher* url_fetcher = url_fetcher_factory_.GetFetcherByID(0);
+ net::TestURLFetcher* url_fetcher = GetRunningFetcher();
DCHECK(url_fetcher);
url_fetcher->set_status(net::URLRequestStatus::FromError(net_error));
url_fetcher->SetResponseString("");
@@ -26,7 +26,7 @@ void PrefetchRequestTestBase::RespondWithNetError(int net_error) {
}
void PrefetchRequestTestBase::RespondWithHttpError(int http_error) {
- net::TestURLFetcher* url_fetcher = url_fetcher_factory_.GetFetcherByID(0);
+ net::TestURLFetcher* url_fetcher = GetRunningFetcher();
DCHECK(url_fetcher);
url_fetcher->set_status(net::URLRequestStatus());
url_fetcher->set_response_code(http_error);
@@ -35,7 +35,7 @@ void PrefetchRequestTestBase::RespondWithHttpError(int http_error) {
}
void PrefetchRequestTestBase::RespondWithData(const std::string& data) {
- net::TestURLFetcher* url_fetcher = url_fetcher_factory_.GetFetcherByID(0);
+ net::TestURLFetcher* url_fetcher = GetRunningFetcher();
DCHECK(url_fetcher);
url_fetcher->set_status(net::URLRequestStatus());
url_fetcher->set_response_code(net::HTTP_OK);
@@ -43,4 +43,9 @@ void PrefetchRequestTestBase::RespondWithData(const std::string& data) {
url_fetcher->delegate()->OnURLFetchComplete(url_fetcher);
}
+net::TestURLFetcher* PrefetchRequestTestBase::GetRunningFetcher() {
+ // All created TestURLFetchers have ID 0 by default.
+ return url_fetcher_factory_.GetFetcherByID(0);
+}
+
} // namespace offline_pages

Powered by Google App Engine
This is Rietveld 408576698