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

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

Issue 8395038: Make test URLFetcher implementations not derive from the URLFetcher implementation, since we want... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: move factory to its own file and remove Create function from URLFetcher impl Created 9 years, 2 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: content/common/net/url_fetcher.cc
===================================================================
--- content/common/net/url_fetcher.cc (revision 107268)
+++ content/common/net/url_fetcher.cc (working copy)
@@ -19,6 +19,7 @@
#include "base/string_util.h"
#include "base/threading/thread.h"
#include "content/public/common/url_fetcher_delegate.h"
+#include "content/public/common/url_fetcher_factory.h"
#include "googleurl/src/gurl.h"
#include "net/base/host_port_pair.h"
#include "net/base/io_buffer.h"
@@ -463,7 +464,7 @@
}
// static
-URLFetcher::Factory* URLFetcher::factory_ = NULL;
+static content::URLFetcherFactory* g_factory = NULL;
wtc 2011/10/26 23:34:46 Remove the "// static" comment on the previous lin
wtc 2011/10/27 21:29:16 Just wondering: why did you change this static mem
jam 2011/10/27 22:34:41 because now I need to use it from the static conte
// static
static bool g_interception_enabled = false;
@@ -482,7 +483,8 @@
const GURL& url,
RequestType request_type,
content::URLFetcherDelegate* d) {
- return ::URLFetcher::Create(id, url, request_type, d);
+ return g_factory ? g_factory->CreateURLFetcher(id, url, request_type, d) :
+ new ::URLFetcher(url, request_type, d);
}
// static
@@ -507,14 +509,6 @@
core_->Stop();
}
-// static
-URLFetcher* URLFetcher::Create(int id, const GURL& url,
- RequestType request_type,
- content::URLFetcherDelegate* d) {
- return factory_ ? factory_->CreateURLFetcher(id, url, request_type, d) :
- new URLFetcher(url, request_type, d);
-}
-
URLFetcher::Core::Core(URLFetcher* fetcher,
const GURL& original_url,
RequestType request_type,
@@ -989,11 +983,6 @@
return core_->backoff_delay_;
}
-void URLFetcher::SetBackoffDelayForTesting(
- base::TimeDelta backoff_delay) {
- core_->backoff_delay_ = backoff_delay;
-}
-
void URLFetcher::SaveResponseToTemporaryFile(
scoped_refptr<base::MessageLoopProxy> file_message_loop_proxy) {
core_->file_message_loop_proxy_ = file_message_loop_proxy;
@@ -1082,16 +1071,6 @@
return true;
}
-void URLFetcher::SetResponseDestinationForTesting(
- ResponseDestinationType value) {
- core_->response_destination_ = value;
-}
-
-URLFetcher::ResponseDestinationType
-URLFetcher::GetResponseDestinationForTesting() const {
- return core_->response_destination_;
-}
-
bool URLFetcher::GetResponseAsFilePath(bool take_ownership,
FilePath* out_response_path) const {
DCHECK(core_->delegate_loop_proxy_->BelongsToCurrentThread());
@@ -1121,3 +1100,13 @@
content::URLFetcherDelegate* URLFetcher::delegate() const {
return core_->delegate();
}
+
+// static
+content::URLFetcherFactory* URLFetcher::factory() {
+ return g_factory;
+}
+
+// static
+void URLFetcher::set_factory(content::URLFetcherFactory* factory) {
+ g_factory = factory;
+}

Powered by Google App Engine
This is Rietveld 408576698