Defer ARC file system operations while ARC is booting.

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

Index: chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_util.h
diff --git a/chrome/browser/chromeos/arc/fileapi/arc_file_system_instance_util.h b/chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_util.h
similarity index 68%
rename from chrome/browser/chromeos/arc/fileapi/arc_file_system_instance_util.h
rename to chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_util.h
index 0ebc74465d4109130e68fbf35e9e065ab22ee3ee..6a2983ff38fcfa45f6860c8968e200f5ed6fd5d1 100644
--- a/chrome/browser/chromeos/arc/fileapi/arc_file_system_instance_util.h
+++ b/chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner_util.h
@@ -2,17 +2,15 @@
 // 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_FILE_SYSTEM_INSTANCE_UTIL_H_
-#define CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_FILE_SYSTEM_INSTANCE_UTIL_H_
+#ifndef CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_FILE_SYSTEM_OPERATION_RUNNER_UTIL_H_
+#define CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_FILE_SYSTEM_OPERATION_RUNNER_UTIL_H_
 
 #include <string>
 
 #include "components/arc/common/file_system.mojom.h"
 
-class GURL;
-
 namespace arc {
-namespace file_system_instance_util {
+namespace file_system_operation_runner_util {
 
 using GetFileSizeCallback = mojom::FileSystemInstance::GetFileSizeCallback;
 using OpenFileToReadCallback =
@@ -21,11 +19,11 @@ using GetDocumentCallback = mojom::FileSystemInstance::GetDocumentCallback;
 using GetChildDocumentsCallback =
     mojom::FileSystemInstance::GetChildDocumentsCallback;
 
-// Utility functions to post a task to run FileSystemInstance methods.
+// Utility functions to post a task to run ArcFileSystemOperationRunner methods.
 // These functions must be called on the IO thread.
-void GetFileSizeOnIOThread(const GURL& arc_url,
+void GetFileSizeOnIOThread(const std::string& url,

[hidehiko, 2017/01/20 08:01:49]

Clarification: Any reasons you change GURL to std::string, while this is still
representing URL?
AFAIK, in Chrome code, practically, GURL is the type to represent URL in most
cases.
If there is no big reason, GURL is preferred, I think.

[Shuhei Takahashi, 2017/01/20 09:13:04]

OK, let's stick with GURL.

                            const GetFileSizeCallback& callback);
-void OpenFileToReadOnIOThread(const GURL& arc_url,
+void OpenFileToReadOnIOThread(const std::string& url,
                               const OpenFileToReadCallback& callback);
 void GetDocumentOnIOThread(const std::string& authority,
                            const std::string& document_id,
@@ -34,7 +32,7 @@ void GetChildDocumentsOnIOThread(const std::string& authority,
                                  const std::string& parent_document_id,
                                  const GetChildDocumentsCallback& callback);
 
-}  // namespace file_system_instance_util
+}  // namespace file_system_operation_runner_util
 }  // namespace arc
 
-#endif  // CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_FILE_SYSTEM_INSTANCE_UTIL_H_
+#endif  // CHROME_BROWSER_CHROMEOS_ARC_FILEAPI_ARC_FILE_SYSTEM_OPERATION_RUNNER_UTIL_H_