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

Unified Diff: webkit/browser/fileapi/file_system_url_request_job_factory.cc

Issue 206253002: Revert of Add mechanism to auto mount file systems in response to a URL request. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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
« no previous file with comments | « webkit/browser/fileapi/file_system_url_request_job_factory.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/browser/fileapi/file_system_url_request_job_factory.cc
diff --git a/webkit/browser/fileapi/file_system_url_request_job_factory.cc b/webkit/browser/fileapi/file_system_url_request_job_factory.cc
index 3add372e5e647eb9413171c6fc53bf021c8a5915..76ae7c9ac0596b801cd778fd31010d4ffcd97dbd 100644
--- a/webkit/browser/fileapi/file_system_url_request_job_factory.cc
+++ b/webkit/browser/fileapi/file_system_url_request_job_factory.cc
@@ -19,8 +19,7 @@
class FileSystemProtocolHandler
: public net::URLRequestJobFactory::ProtocolHandler {
public:
- FileSystemProtocolHandler(const std::string& storage_domain,
- FileSystemContext* context);
+ explicit FileSystemProtocolHandler(FileSystemContext* context);
virtual ~FileSystemProtocolHandler();
virtual net::URLRequestJob* MaybeCreateJob(
@@ -28,8 +27,6 @@
net::NetworkDelegate* network_delegate) const OVERRIDE;
private:
- const std::string storage_domain_;
-
// No scoped_refptr because |file_system_context_| is owned by the
// ProfileIOData, which also owns this ProtocolHandler.
FileSystemContext* const file_system_context_;
@@ -38,10 +35,8 @@
};
FileSystemProtocolHandler::FileSystemProtocolHandler(
- const std::string& storage_domain,
FileSystemContext* context)
- : storage_domain_(storage_domain),
- file_system_context_(context) {
+ : file_system_context_(context) {
DCHECK(file_system_context_);
}
@@ -56,18 +51,18 @@
// redirects back here, by adding a / to the URL.
if (!path.empty() && path[path.size() - 1] == '/') {
return new FileSystemDirURLRequestJob(
- request, network_delegate, storage_domain_, file_system_context_);
+ request, network_delegate, file_system_context_);
}
return new FileSystemURLRequestJob(
- request, network_delegate, storage_domain_, file_system_context_);
+ request, network_delegate, file_system_context_);
}
} // anonymous namespace
-net::URLRequestJobFactory::ProtocolHandler* CreateFileSystemProtocolHandler(
- const std::string& storage_domain, FileSystemContext* context) {
+net::URLRequestJobFactory::ProtocolHandler*
+CreateFileSystemProtocolHandler(FileSystemContext* context) {
DCHECK(context);
- return new FileSystemProtocolHandler(storage_domain, context);
+ return new FileSystemProtocolHandler(context);
}
} // namespace fileapi
« no previous file with comments | « webkit/browser/fileapi/file_system_url_request_job_factory.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698