mediaview: Introduce ArcDocumentsProviderRoot.

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

Index: chrome/browser/chromeos/arc/fileapi/arc_documents_provider_async_file_util.h
diff --git a/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_async_file_util.h b/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_async_file_util.h
index 24037e2fc0a6353f74c26dab893cf4d5a66e0b81..f0334efbc98a6fc44ea0718e2bcd463c5bf2b803 100644
--- a/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_async_file_util.h
+++ b/chrome/browser/chromeos/arc/fileapi/arc_documents_provider_async_file_util.h
@@ -9,16 +9,20 @@
 
 #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.
@@ -89,6 +93,20 @@ class ArcDocumentsProviderAsyncFileUtil : public storage::AsyncFileUtil {
       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);
 };