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

Unified Diff: net/url_request/file_protocol_handler.h

Issue 22795006: Remove WorkerPool dependency from URLRequestFileJob. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix android include Created 7 years, 4 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/file_protocol_handler.h
diff --git a/net/url_request/file_protocol_handler.h b/net/url_request/file_protocol_handler.h
index 8087a6ee5a9ea7e85b9fe26b7cbfdb5bf66c065c..5c103c25bbda85126f3ee299e02779fe829e67a7 100644
--- a/net/url_request/file_protocol_handler.h
+++ b/net/url_request/file_protocol_handler.h
@@ -7,6 +7,8 @@
#include "base/basictypes.h"
#include "base/compiler_specific.h"
+#include "base/memory/ref_counted.h"
+#include "base/task_runner.h"
#include "net/url_request/url_request_job_factory.h"
class GURL;
@@ -21,12 +23,14 @@ class URLRequestJob;
class NET_EXPORT FileProtocolHandler :
public URLRequestJobFactory::ProtocolHandler {
public:
- FileProtocolHandler();
+ FileProtocolHandler(const scoped_refptr<base::TaskRunner>& task_runner);
akalin 2013/08/15 17:43:05 explicit (also, all other similar places)
akalin 2013/08/15 17:43:05 can you name this something other than just task_r
+ virtual ~FileProtocolHandler();
virtual URLRequestJob* MaybeCreateJob(
URLRequest* request, NetworkDelegate* network_delegate) const OVERRIDE;
virtual bool IsSafeRedirectTarget(const GURL& location) const OVERRIDE;
private:
+ scoped_refptr<base::TaskRunner> task_runner_;
akalin 2013/08/15 17:43:05 const (also, all other similar places)
DISALLOW_COPY_AND_ASSIGN(FileProtocolHandler);
};

Powered by Google App Engine
This is Rietveld 408576698