Move more utility functions to arc_util.

chrome/browser/ui/extensions/app_launch_params.cc

Index: chrome/browser/ui/extensions/app_launch_params.cc
diff --git a/chrome/browser/ui/extensions/app_launch_params.cc b/chrome/browser/ui/extensions/app_launch_params.cc
index bc863c4fa2871ca3ed173612e284ddc5838367e8..dc4f1bd55739c6f0065851e0c13e432febc52b87 100644
--- a/chrome/browser/ui/extensions/app_launch_params.cc
+++ b/chrome/browser/ui/extensions/app_launch_params.cc
@@ -13,8 +13,8 @@
 #include "ui/base/window_open_disposition.h"
 
 #if defined(OS_CHROMEOS)
-#include "chrome/browser/chromeos/arc/arc_session_manager.h"
-#include "components/arc/arc_bridge_service.h"
+#include "chrome/browser/chromeos/arc/arc_util.h"
+#include "components/arc/arc_util.h"
 #endif
 
 using extensions::ExtensionPrefs;
@@ -35,10 +35,9 @@ AppLaunchParams::AppLaunchParams(Profile* profile,
       play_store_status(PlayStoreStatus::PLAY_STORE_STATUS_UNKNOWN) {
 #if defined(OS_CHROMEOS)
   if (set_playstore_status) {
-    if (arc::ArcSessionManager::IsAllowedForProfile(profile)) {
+    if (arc::util::IsArcAllowedForProfile(profile)) {

[Luis Héctor Chávez, 2017/01/19 18:09:15]

nit: maybe elide braces? Not sure how that plays with the comment in L42. Up to
you.

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

I couldn't find any guideline, and I think I've rarely seen dropping braces in
chromium if there is else clause (though, I do not have stats...). For now, let
me keep this as is.

       play_store_status = PlayStoreStatus::PLAY_STORE_STATUS_ENABLED;
-    } else if (arc::ArcBridgeService::GetAvailable(
-                   base::CommandLine::ForCurrentProcess())) {
+    } else if (arc::util::IsArcAvailable()) {
       play_store_status = PlayStoreStatus::PLAY_STORE_STATUS_AVAILABLE;
     }  // else, default to PLAY_STORE_STATUS_UNKNOWN.
   }