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

Unified Diff: chrome/browser/ui/app_list/arc/arc_app_utils.cc

Issue 2898273002: Move Voice Interaction shortcut to accelerator_controller.cc (Closed)
Patch Set: Refactor Created 3 years, 7 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/app_list/arc/arc_app_utils.cc
diff --git a/chrome/browser/ui/app_list/arc/arc_app_utils.cc b/chrome/browser/ui/app_list/arc/arc_app_utils.cc
index 799794407cd791189f7b47479a7fc99b8296902c..e8105a21fcd2522ed5dd8245c4e9820fb4c7ff62 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_utils.cc
+++ b/chrome/browser/ui/app_list/arc/arc_app_utils.cc
@@ -69,12 +69,6 @@ constexpr char kSetInTouchModeIntent[] =
constexpr char kShowTalkbackSettingsIntent[] =
"org.chromium.arc.intent_helper.SHOW_TALKBACK_SETTINGS";
-void SetArcCpuRestrictionCallback(bool success) {
- VLOG(2) << "Finished prioritizing the instance: result=" << success;
- if (!success)
- LOG(ERROR) << "Failed to prioritize ARC";
-}
-
// Find a proper size and position for a given rectangle on the screen.
// TODO(skuhne): This needs more consideration, but it is lacking
// WindowPositioner functionality since we do not have an Aura::Window yet.
@@ -335,12 +329,7 @@ bool LaunchApp(content::BrowserContext* context,
// default to avoid slowing down Chrome's user session restoration.
// However, the restriction should be lifted once the user explicitly
// tries to launch an ARC app.
- VLOG(2) << "Prioritizing the instance";
- chromeos::SessionManagerClient* session_manager_client =
- chromeos::DBusThreadManager::Get()->GetSessionManagerClient();
- session_manager_client->SetArcCpuRestriction(
- login_manager::CONTAINER_CPU_RESTRICTION_FOREGROUND,
- base::Bind(SetArcCpuRestrictionCallback));
+ PrioritizeArcContainerStartup();
}
prefs->SetLastLaunchTime(app_id, base::Time::Now());
return true;

Powered by Google App Engine
This is Rietveld 408576698