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

Unified Diff: chrome/browser/chromeos/file_system_provider/provided_file_system.h

Issue 210803003: [fsp] Decouple file_service_provider::Service. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments. Created 6 years, 8 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/chromeos/file_system_provider/provided_file_system.h
diff --git a/chrome/browser/chromeos/file_system_provider/provided_file_system.h b/chrome/browser/chromeos/file_system_provider/provided_file_system.h
index 10aa0a93fc291a589c2f2adb75a7ab7ec7257b5b..45807a45618ac6c5667207ec2db181783b91c83f 100644
--- a/chrome/browser/chromeos/file_system_provider/provided_file_system.h
+++ b/chrome/browser/chromeos/file_system_provider/provided_file_system.h
@@ -5,41 +5,38 @@
#ifndef CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_PROVIDED_FILE_SYSTEM_H_
#define CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_PROVIDED_FILE_SYSTEM_H_
-#include <string>
+#include "chrome/browser/chromeos/file_system_provider/provided_file_system_info.h"
+#include "chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h"
-#include "base/files/file_path.h"
+namespace extensions {
+class EventRouter;
+} // namespace extensions
namespace chromeos {
namespace file_system_provider {
-// Contains information about the provided file system instance.
-class ProvidedFileSystem {
- public:
- ProvidedFileSystem();
- ProvidedFileSystem(const std::string& extension_id,
- int file_system_id,
- const std::string& file_system_name,
- const base::FilePath& mount_path);
+class RequestManager;
- ~ProvidedFileSystem();
+// Provided file system implementation. Forwards requests between providers and
+// clients.
+class ProvidedFileSystem : public ProvidedFileSystemInterface {
+ public:
+ ProvidedFileSystem(extensions::EventRouter* event_router,
+ RequestManager* request_manager,
+ const ProvidedFileSystemInfo& file_system_info);
+ virtual ~ProvidedFileSystem();
- const std::string& extension_id() const { return extension_id_; }
- int file_system_id() const { return file_system_id_; }
- const std::string& file_system_name() const { return file_system_name_; }
- const base::FilePath& mount_path() const { return mount_path_; }
+ // ProvidedFileSystemInterface overrides.
+ virtual bool RequestUnmount(
+ const fileapi::AsyncFileUtil::StatusCallback& callback) OVERRIDE;
+ virtual const ProvidedFileSystemInfo& GetFileSystemInfo() const OVERRIDE;
private:
- // ID of the extension providing this file system.
- std::string extension_id_;
-
- // ID of the file system, used internally.
- int file_system_id_;
-
- // Name of the file system, can be rendered in the UI.
- std::string file_system_name_;
+ extensions::EventRouter* event_router_;
+ RequestManager* request_manager_;
+ ProvidedFileSystemInfo file_system_info_;
- // Mount path of the underlying file system.
- base::FilePath mount_path_;
+ DISALLOW_COPY_AND_ASSIGN(ProvidedFileSystem);
};
} // namespace file_system_provider

Powered by Google App Engine
This is Rietveld 408576698