mediaview: Fix ARC file system operation deferring.

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

Index: chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.h
diff --git a/chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.h b/chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.h
index 958e46254a3e4a949b55a9432ff0a3b9013899ae..e1b32569a8f717cf4ba52c209c86494898e97937 100644
--- a/chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.h
+++ b/chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.h
@@ -66,7 +66,8 @@ class ArcFileSystemOperationRunner
 
   // The standard constructor. A production instance should be created by
   // this constructor.
-  explicit ArcFileSystemOperationRunner(ArcBridgeService* bridge_service);
+  ArcFileSystemOperationRunner(ArcBridgeService* bridge_service,
+                               const Profile* profile);
   ~ArcFileSystemOperationRunner() override;
 
   // Runs file system operations. See file_system.mojom for documentation.
@@ -90,6 +91,7 @@ class ArcFileSystemOperationRunner
   friend class ArcFileSystemOperationRunnerTest;
 
   ArcFileSystemOperationRunner(ArcBridgeService* bridge_service,
+                               const Profile* profile,
                                bool observe_events);
 
   // Called whenever ARC states related to |should_defer_| are changed.
@@ -100,6 +102,9 @@ class ArcFileSystemOperationRunner
   // deferring.
   void SetShouldDefer(bool should_defer);
 
+  // Profile this runner is associated with.

[hidehiko, 2017/02/23 08:36:04]

Could you note that this can be null for testing?

[Shuhei Takahashi, 2017/02/23 08:49:30]

Done.

+  const Profile* profile_;

[hidehiko, 2017/02/23 08:36:04]

const Profile* const profile_;

In ARC, it is encouraged to make this "const" if it is not changed in the class.
You can find it in ARC style guide.

[Shuhei Takahashi, 2017/02/23 08:49:30]

I agree we want more const :)
Also changed observe_events_.

+
   // Indicates if this instance should register observers to receive events.
   // Usually true, but set to false in unit tests.
   bool observe_events_;