Move more utility functions to arc_util.

chrome/browser/chromeos/arc/arc_session_manager.h

Index: chrome/browser/chromeos/arc/arc_session_manager.h
diff --git a/chrome/browser/chromeos/arc/arc_session_manager.h b/chrome/browser/chromeos/arc/arc_session_manager.h
index 20ef5ab3104ee0b42a0e0d9229b490414b3d85fe..c5b70b7fe358addb19622c4c25ca1fd5b0a00d56 100644
--- a/chrome/browser/chromeos/arc/arc_session_manager.h
+++ b/chrome/browser/chromeos/arc/arc_session_manager.h
@@ -126,18 +126,8 @@ class ArcSessionManager : public ArcSessionObserver,
 
   static void DisableUIForTesting();
   static void SetShelfDelegateForTesting(ash::ShelfDelegate* shelf_delegate);
-
-  // Checks if OptIn verification was disabled by switch in command line.
-  static bool IsOptInVerificationDisabled();
-
   static void EnableCheckAndroidManagementForTesting();
 
-  // Returns true if Arc is allowed to run for the given profile.
-  static bool IsAllowedForProfile(const Profile* profile);
-
-  // Returns true if ARC should run under Kiosk mode.
-  static bool IsArcKioskMode();
-
   // Returns true if Arc is allowed to run for the current session.
   bool IsAllowed() const;

[Yusuke Sato, 2017/01/19 22:39:31]

I feel the function name is too similar to IsAllowedForProfile() in
c/b/c/a/arc_util.h while this one only checks if profile_ is null. Can't we
rename this to something like IsPrimaryUserProfileReady() and update the comment
at L131?

Doing this in a separate CL is fine with a TODO.

[hidehiko, 2017/01/24 17:46:11]

Sure. Added TODO.