Extract kArcEnabled preference from ArcSessionManager part 1.

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

 // Copyright 2017 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.
 
 #include "chrome/browser/chromeos/arc/fileapi/arc_file_system_operation_runner.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/memory/ptr_util.h"
 #include "base/optional.h"
 #include "base/threading/thread_task_runner_handle.h"
+#include "chrome/browser/chromeos/arc/arc_util.h"
 #include "components/arc/arc_bridge_service.h"
 #include "content/public/browser/browser_thread.h"
 #include "url/gurl.h"
 
 using content::BrowserThread;
 
 namespace arc {
 
 // static
 const char ArcFileSystemOperationRunner::kArcServiceName[] =
@@ skipping 132 matching lines @@
   OnStateChanged();
 }
 
 void ArcFileSystemOperationRunner::OnInstanceClosed() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   OnStateChanged();
 }
 
 void ArcFileSystemOperationRunner::OnStateChanged() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  SetShouldDefer(ArcSessionManager::Get()->IsArcPlayStoreEnabled() &&
-                 !arc_bridge_service()->file_system()->has_instance());
+  // TODO(hidehiko): Revisit the condition, when ARC is running without
+  // profile.
+  SetShouldDefer(

[hidehiko, 2017/02/17 15:07:19]

FYI (off-topic): yusukes@.
On ARC login screen work, this may be considered.
This module conceptually expects that ARC is tied to primary Profile, but on
login screen, primary profile is not prepared.

[Yusuke Sato, 2017/02/17 22:27:09]

Acknowledged.

+      IsArcPlayStoreEnabledForProfile(ArcSessionManager::Get()->profile()) &&
+      !arc_bridge_service()->file_system()->has_instance());
 }
 
 void ArcFileSystemOperationRunner::SetShouldDefer(bool should_defer) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   should_defer_ = should_defer;
 
   if (should_defer_)
     return;
 
   // Run deferred operations.
   std::vector<base::Closure> deferred_operations;
   deferred_operations.swap(deferred_operations_);
   for (const base::Closure& operation : deferred_operations) {
     operation.Run();
   }
 
   // No deferred operations should be left at this point.
   DCHECK(deferred_operations_.empty());
 }
 
 }  // namespace arc