mojo filesystem: Further clean ups.

components/filesystem/file_system_impl.h

Index: components/filesystem/file_system_impl.h
diff --git a/components/filesystem/file_system_impl.h b/components/filesystem/file_system_impl.h
index 4c18dd562cfe39aff61b94c69f98275f62e6f8d7..df796e29d9e59402bab2a6a9dfd103507d23ae38 100644
--- a/components/filesystem/file_system_impl.h
+++ b/components/filesystem/file_system_impl.h
@@ -5,6 +5,7 @@
 #ifndef COMPONENTS_FILESYSTEM_FILE_SYSTEM_IMPL_H_
 #define COMPONENTS_FILESYSTEM_FILE_SYSTEM_IMPL_H_
 
+#include "base/files/file_path.h"
 #include "base/macros.h"
 #include "components/filesystem/public/interfaces/file_system.mojom.h"
 #include "mojo/public/cpp/bindings/interface_request.h"
@@ -23,38 +24,30 @@ class FileSystemApp;
 
 class LockTable;
 
+// The base implementation of FileSystemImpl.
 class FileSystemImpl : public FileSystem {
  public:
   FileSystemImpl(mojo::Connection* connection,
                  mojo::InterfaceRequest<FileSystem> request,
+                 base::FilePath user_data_dir,

[jam, 2016/02/23 16:51:25]

nit: add comment to this constructor explaining the arguments.

in particular, user_data_dir should be explained and probably renamed. since
this is a generic component, it seems like it shouldn't know about profiles and
user data dir. can it just be "directory"?

                  LockTable* lock_table);
   ~FileSystemImpl() override;
 
   // |Files| implementation:
-
-  // Current valid values for |file_system| are "temp" for a temporary
-  // filesystem and "origin" for a persistent filesystem bound to the origin of
-  // the URL of the caller.
-  void OpenFileSystem(const mojo::String& file_system,
-                      mojo::InterfaceRequest<Directory> directory,
-                      FileSystemClientPtr client,
-                      const OpenFileSystemCallback& callback) override;
+  void OpenTempDirectory(
+      mojo::InterfaceRequest<Directory> directory,
+      const OpenTempDirectoryCallback& callback) override;
+  void OpenPersistentFileSystem(
+      mojo::InterfaceRequest<Directory> directory,
+      const OpenPersistentFileSystemCallback& callback) override;
 
  private:
-  // Gets the system specific toplevel profile directory.
-  base::FilePath GetSystemProfileDir() const;
-
-  // Takes the origin string from |remote_application_url_|.
-  std::string GetOriginFromRemoteApplicationURL() const;
-
-  // Sanitizes |origin| so it is an acceptable filesystem name.
-  void BuildSanitizedOrigin(const std::string& origin,
-                            std::string* sanitized_origin);
-
   const std::string remote_application_url_;
   mojo::StrongBinding<FileSystem> binding_;
   LockTable* lock_table_;
 
+  base::FilePath user_data_dir_;
+
   DISALLOW_COPY_AND_ASSIGN(FileSystemImpl);
 };