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

Unified Diff: chrome/browser/net/file_system_url_request_job_factory.cc

Issue 7036004: Remove more URLRequest::RegisterProtocolFactory calls. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup nits. Created 9 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: chrome/browser/net/file_system_url_request_job_factory.cc
diff --git a/chrome/browser/net/file_system_url_request_job_factory.cc b/chrome/browser/net/file_system_url_request_job_factory.cc
deleted file mode 100644
index ba7b1aebdcbbd7776fd88ac452e2729442bb85d0..0000000000000000000000000000000000000000
--- a/chrome/browser/net/file_system_url_request_job_factory.cc
+++ /dev/null
@@ -1,66 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/net/file_system_url_request_job_factory.h"
-
-#include "base/basictypes.h"
-#include "base/compiler_specific.h"
-#include "chrome/browser/net/chrome_url_request_context.h"
-#include "chrome/common/url_constants.h"
-#include "content/browser/browser_thread.h"
-#include "net/url_request/url_request.h"
-#include "webkit/fileapi/file_system_url_request_job.h"
-#include "webkit/fileapi/file_system_dir_url_request_job.h"
-
-namespace {
-
-class FileSystemProtocolHandler
- : public net::URLRequestJobFactory::ProtocolHandler {
- public:
- explicit FileSystemProtocolHandler(fileapi::FileSystemContext* context);
- virtual ~FileSystemProtocolHandler();
-
- virtual net::URLRequestJob* MaybeCreateJob(
- net::URLRequest* request) const OVERRIDE;
-
- private:
- // No scoped_refptr because |file_system_context_| is owned by the
- // ProfileIOData, which also owns this ProtocolHandler.
- fileapi::FileSystemContext* const file_system_context_;
-
- DISALLOW_COPY_AND_ASSIGN(FileSystemProtocolHandler);
-};
-
-FileSystemProtocolHandler::FileSystemProtocolHandler(
- fileapi::FileSystemContext* context)
- : file_system_context_(context) {
- DCHECK(file_system_context_);
-}
-
-FileSystemProtocolHandler::~FileSystemProtocolHandler() {}
-
-net::URLRequestJob* FileSystemProtocolHandler::MaybeCreateJob(
- net::URLRequest* request) const {
- const std::string path = request->url().path();
-
- // If the path ends with a /, we know it's a directory. If the path refers
- // 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 fileapi::FileSystemDirURLRequestJob(
- request, file_system_context_,
- BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE));
- }
- return new fileapi::FileSystemURLRequestJob(
- request, file_system_context_,
- BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE));
-}
-
-} // anonymous namespace
-
-net::URLRequestJobFactory::ProtocolHandler*
-CreateFileSystemProtocolHandler(fileapi::FileSystemContext* context) {
- DCHECK(context);
- return new FileSystemProtocolHandler(context);
-}
« no previous file with comments | « chrome/browser/net/file_system_url_request_job_factory.h ('k') | chrome/browser/profiles/profile_io_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698