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

Unified Diff: webkit/fileapi/file_system_url_request_job_factory.cc

Issue 10197007: Change webkit/{fileapi,quota} code to use TaskRunner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: test fix Created 8 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: webkit/fileapi/file_system_url_request_job_factory.cc
diff --git a/webkit/fileapi/file_system_url_request_job_factory.cc b/webkit/fileapi/file_system_url_request_job_factory.cc
index 4e5855a1b67ac5a951c953296abc451bb838e2af..ae6f39ca39c55492869c59f9bbd3f0d617190146 100644
--- a/webkit/fileapi/file_system_url_request_job_factory.cc
+++ b/webkit/fileapi/file_system_url_request_job_factory.cc
@@ -19,9 +19,7 @@ namespace {
class FileSystemProtocolHandler
: public net::URLRequestJobFactory::ProtocolHandler {
public:
- explicit FileSystemProtocolHandler(
- FileSystemContext* context,
- base::MessageLoopProxy* loop_proxy);
+ explicit FileSystemProtocolHandler(FileSystemContext* context);
virtual ~FileSystemProtocolHandler();
virtual net::URLRequestJob* MaybeCreateJob(
@@ -31,18 +29,14 @@ class FileSystemProtocolHandler
// No scoped_refptr because |file_system_context_| is owned by the
// ProfileIOData, which also owns this ProtocolHandler.
FileSystemContext* const file_system_context_;
- const scoped_refptr<base::MessageLoopProxy> file_loop_proxy_;
DISALLOW_COPY_AND_ASSIGN(FileSystemProtocolHandler);
};
FileSystemProtocolHandler::FileSystemProtocolHandler(
- FileSystemContext* context,
- base::MessageLoopProxy* file_loop_proxy)
- : file_system_context_(context),
- file_loop_proxy_(file_loop_proxy) {
+ FileSystemContext* context)
+ : file_system_context_(context) {
DCHECK(file_system_context_);
- DCHECK(file_loop_proxy_);
}
FileSystemProtocolHandler::~FileSystemProtocolHandler() {}
@@ -55,21 +49,17 @@ net::URLRequestJob* FileSystemProtocolHandler::MaybeCreateJob(
// to a directory and gets dispatched to FileSystemURLRequestJob, that class
// redirects back here, by adding a / to the URL.
if (!path.empty() && path[path.size() - 1] == '/') {
- return new FileSystemDirURLRequestJob(
- request, file_system_context_, file_loop_proxy_);
+ return new FileSystemDirURLRequestJob(request, file_system_context_);
}
- return new FileSystemURLRequestJob(
- request, file_system_context_, file_loop_proxy_);
+ return new FileSystemURLRequestJob(request, file_system_context_);
}
} // anonymous namespace
net::URLRequestJobFactory::ProtocolHandler*
-CreateFileSystemProtocolHandler(FileSystemContext* context,
- base::MessageLoopProxy* loop_proxy) {
+CreateFileSystemProtocolHandler(FileSystemContext* context) {
DCHECK(context);
- DCHECK(loop_proxy);
- return new FileSystemProtocolHandler(context, loop_proxy);
+ return new FileSystemProtocolHandler(context);
}
} // namespace fileapi
« no previous file with comments | « webkit/fileapi/file_system_url_request_job_factory.h ('k') | webkit/fileapi/file_system_url_request_job_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698