mediaview: Introduce ArcDocumentsProviderRoot.

chrome/browser/chromeos/arc/fileapi/arc_documents_provider_async_file_util.h

 // Copyright 2016 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_ARC_FILEAPI_ARC_DOCUMENTS_PROVIDER_ASYNC_FILE_UTIL_H_
 #define CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_DOCUMENTS_PROVIDER_ASYNC_FILE_UTIL_H_
 
 #include <memory>
 
 #include "base/callback.h"
 #include "base/macros.h"
+#include "base/memory/weak_ptr.h"
 #include "storage/browser/fileapi/async_file_util.h"
 
 namespace arc {
 
+class ArcDocumentsProviderRootMap;
+
 // The implementation of storage::AsyncFileUtil for media view.
 //
 // All of the methods must be called on the IO thread.
 class ArcDocumentsProviderAsyncFileUtil : public storage::AsyncFileUtil {
  public:
-  ArcDocumentsProviderAsyncFileUtil();
+  explicit ArcDocumentsProviderAsyncFileUtil(
+      ArcDocumentsProviderRootMap* roots);
   ~ArcDocumentsProviderAsyncFileUtil() override;
 
   // storage::AsyncFileUtil overrides.
   void CreateOrOpen(
       std::unique_ptr<storage::FileSystemOperationContext> context,
       const storage::FileSystemURL& url,
       int file_flags,
       const CreateOrOpenCallback& callback) override;
   void EnsureFileExists(
       std::unique_ptr<storage::FileSystemOperationContext> context,
@@ skipping 50 matching lines @@
   void DeleteRecursively(
       std::unique_ptr<storage::FileSystemOperationContext> context,
       const storage::FileSystemURL& url,
       const StatusCallback& callback) override;
   void CreateSnapshotFile(
       std::unique_ptr<storage::FileSystemOperationContext> context,
       const storage::FileSystemURL& url,
       const CreateSnapshotFileCallback& callback) override;
 
  private:
+  void OnGetFileInfo(
+      std::unique_ptr<storage::FileSystemOperationContext> context,
+      const GetFileInfoCallback& callback,
+      base::File::Error result,
+      const base::File::Info& file_info);
+  void OnReadDirectory(storage::FileSystemOperationContext* context,
+                       const ReadDirectoryCallback& callback,
+                       base::File::Error result,
+                       const EntryList& file_list,
+                       bool has_more);
+
+  ArcDocumentsProviderRootMap* roots_;

[Luis Héctor Chávez, 2016/12/14 20:53:50]

Please document who owns this pointer. Also, it seems to be possible to make
this 

ArcDocumentsProviderRootMap* const roots_;

[Shuhei Takahashi, 2016/12/15 02:37:55]

Done.

+  base::WeakPtrFactory<ArcDocumentsProviderAsyncFileUtil> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(ArcDocumentsProviderAsyncFileUtil);
 };
 
 }  // namespace arc
 
 #endif  // CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_DOCUMENTS_PROVIDER_ASYNC_FILE_UTIL_H_