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

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: Move Voice Interaction shortcut to accelerator_controller.cc 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 6c019b755eec801119e390b183f8c3834b385de7..8283c65f06fa6ce00df161748b84797409bad966 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_utils.cc
+++ b/chrome/browser/ui/app_list/arc/arc_app_utils.cc
@@ -21,8 +21,6 @@
#include "chrome/browser/ui/ash/launcher/arc_app_deferred_launcher_controller.h"
#include "chrome/browser/ui/ash/launcher/arc_app_shelf_id.h"
#include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
-#include "chromeos/dbus/dbus_thread_manager.h"
-#include "chromeos/dbus/session_manager_client.h"
#include "components/arc/arc_bridge_service.h"
#include "components/arc/arc_features.h"
#include "components/arc/arc_service_manager.h"
@@ -69,12 +67,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 +327,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));
+ PrioritizeArcContainer();
}
prefs->SetLastLaunchTime(app_id, base::Time::Now());
return true;

Powered by Google App Engine
This is Rietveld 408576698