arc: Register ARC content file system

chrome/browser/chromeos/fileapi/file_system_backend.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_FILEAPI_FILE_SYSTEM_BACKEND_H_
 #define CHROME_BROWSER_CHROMEOS_FILEAPI_FILE_SYSTEM_BACKEND_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 54 matching lines @@
 //
 class FileSystemBackend : public storage::ExternalFileSystemBackend {
  public:
   using storage::FileSystemBackend::OpenFileSystemCallback;
 
   // FileSystemBackend will take an ownership of a |mount_points|
   // reference. On the other hand, |system_mount_points| will be kept as a raw
   // pointer and it should outlive FileSystemBackend instance.
   // The ownerships of |drive_delegate| and |file_system_provider_delegate| are
   // also taken.
-  FileSystemBackend(
-      FileSystemBackendDelegate* drive_delegate,
-      FileSystemBackendDelegate* file_system_provider_delegate,
-      FileSystemBackendDelegate* mtp_delegate,
-      scoped_refptr<storage::ExternalMountPoints> mount_points,
-      storage::ExternalMountPoints* system_mount_points);
+  FileSystemBackend(FileSystemBackendDelegate* drive_delegate,

[Luis Héctor Chávez, 2016/10/25 17:15:47]

Can you clearly document (in code) what pointers will be owned by making them
std::unique_ptr<>?

[hashimoto, 2016/10/26 05:23:12]

Done.

+                    FileSystemBackendDelegate* file_system_provider_delegate,
+                    FileSystemBackendDelegate* mtp_delegate,
+                    FileSystemBackendDelegate* arc_content_delegate,
+                    scoped_refptr<storage::ExternalMountPoints> mount_points,
+                    storage::ExternalMountPoints* system_mount_points);
   ~FileSystemBackend() override;
 
   // Adds system mount points, such as "archive", and "removable". This
   // function is no-op if these mount points are already present.
   void AddSystemMountPoints();
 
   // Returns true if CrosMountpointProvider can handle |url|, i.e. its
   // file system type matches with what this provider supports.
   // This could be called on any threads.
   static bool CanHandleURL(const storage::FileSystemURL& url);
@@ skipping 57 matching lines @@
 
   // The delegate instance for the drive file system related operations.
   std::unique_ptr<FileSystemBackendDelegate> drive_delegate_;
 
   // The delegate instance for the provided file system related operations.
   std::unique_ptr<FileSystemBackendDelegate> file_system_provider_delegate_;
 
   // The delegate instance for the MTP file system related operations.
   std::unique_ptr<FileSystemBackendDelegate> mtp_delegate_;
 
+  // The delegate instance for the ARC content file system related operations.
+  std::unique_ptr<FileSystemBackendDelegate> arc_content_delegate_;
+
   // Mount points specific to the owning context (i.e. per-profile mount
   // points).
   //
   // It is legal to have mount points with the same name as in
   // system_mount_points_. Also, mount point paths may overlap with mount point
   // paths in system_mount_points_. In both cases mount points in
   // |mount_points_| will have a priority.
   // E.g. if |mount_points_| map 'foo1' to '/foo/foo1' and
   // |file_system_mount_points_| map 'xxx' to '/foo/foo1/xxx', |GetVirtualPaths|
   // will resolve '/foo/foo1/xxx/yyy' as 'foo1/xxx/yyy' (i.e. the mapping from
   // |mount_points_| will be used).
   scoped_refptr<storage::ExternalMountPoints> mount_points_;
 
   // Globally visible mount points. System MountPonts instance should outlive
   // all FileSystemBackend instances, so raw pointer is safe.
   storage::ExternalMountPoints* system_mount_points_;
 
   DISALLOW_COPY_AND_ASSIGN(FileSystemBackend);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_FILEAPI_FILE_SYSTEM_BACKEND_H_