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

Unified Diff: net/url_request/test_url_fetcher_factory.cc

Issue 23625015: Consolidate TestURLFetcherFactory::SetFakeResponse (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Could have spelled it Cloud... Created 7 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: net/url_request/test_url_fetcher_factory.cc
diff --git a/net/url_request/test_url_fetcher_factory.cc b/net/url_request/test_url_fetcher_factory.cc
index 30380354b2da0a52a657ccb87afcd24a9be8feef..1380c6738d771902fba375d6871ea2263cc7568b 100644
--- a/net/url_request/test_url_fetcher_factory.cc
+++ b/net/url_request/test_url_fetcher_factory.cc
@@ -357,7 +357,7 @@ URLFetcher* FakeURLFetcherFactory::CreateURLFetcher(
return fake_fetcher.release();
}
-void FakeURLFetcherFactory::SetFakeResponseForURL(
+void FakeURLFetcherFactory::SetFakeResponse(
const GURL& url,
const std::string& response_data,
bool success) {
@@ -365,12 +365,6 @@ void FakeURLFetcherFactory::SetFakeResponseForURL(
fake_responses_[url] = std::make_pair(response_data, success);
}
-void FakeURLFetcherFactory::SetFakeResponse(const std::string& url,
- const std::string& response_data,
- bool success) {
- SetFakeResponseForURL(GURL(url), response_data, success);
-}
-
void FakeURLFetcherFactory::ClearFakeResponses() {
fake_responses_.clear();
}

Powered by Google App Engine
This is Rietveld 408576698