Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1511)

Unified Diff: chrome/browser/ui/extensions/app_launch_params.cc

Issue 2648213004: Migrate --enable-arc and --arc-available part 1. (Closed)
Patch Set: Address comments. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..35d3a11c670dd415171c89e372d66803479638dd 100644
--- a/chrome/browser/ui/extensions/app_launch_params.cc
+++ b/chrome/browser/ui/extensions/app_launch_params.cc
@@ -14,7 +14,7 @@
#if defined(OS_CHROMEOS)
#include "chrome/browser/chromeos/arc/arc_session_manager.h"
-#include "components/arc/arc_bridge_service.h"
+#include "components/arc/arc_util.h"
#endif
using extensions::ExtensionPrefs;
@@ -34,11 +34,11 @@ AppLaunchParams::AppLaunchParams(Profile* profile,
source(source),
play_store_status(PlayStoreStatus::PLAY_STORE_STATUS_UNKNOWN) {
#if defined(OS_CHROMEOS)
+ // TODO(b/34478891): Remove this from app launch.
if (set_playstore_status) {
if (arc::ArcSessionManager::IsAllowedForProfile(profile)) {
play_store_status = PlayStoreStatus::PLAY_STORE_STATUS_ENABLED;
- } else if (arc::ArcBridgeService::GetAvailable(
- base::CommandLine::ForCurrentProcess())) {
+ } else if (arc::IsArcAvailable()) {
Yusuke Sato 2017/01/24 19:02:44 nit: no }{
hidehiko 2017/01/25 17:54:46 Hmm... I couldn't find a good example to omit in t
play_store_status = PlayStoreStatus::PLAY_STORE_STATUS_AVAILABLE;
} // else, default to PLAY_STORE_STATUS_UNKNOWN.
}

Powered by Google App Engine
This is Rietveld 408576698