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

Unified Diff: chrome/browser/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc

Issue 2624513004: Prioritize ARC instance only when it is needed (Closed)
Patch Set: review 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/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc
diff --git a/components/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc b/chrome/browser/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc
similarity index 79%
rename from components/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc
rename to chrome/browser/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc
index 79a3e7d7e4ea51e109cb385ee0fdee276a4259de..458e823042e212fdafd7e11ee45f463e70d0bcb8 100644
--- a/components/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc
+++ b/chrome/browser/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.cc
@@ -2,25 +2,16 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "components/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.h"
+#include "chrome/browser/chromeos/arc/boot_phase_monitor/arc_boot_phase_monitor_bridge.h"
#include "base/logging.h"
+#include "chrome/browser/chromeos/arc/boot_phase_monitor/arc_instance_throttle.h"
#include "chromeos/dbus/dbus_thread_manager.h"
#include "chromeos/dbus/session_manager_client.h"
#include "components/arc/arc_bridge_service.h"
namespace arc {
-namespace {
-
-void PrioritizeArcInstanceCallback(bool success) {
- VLOG(2) << "Finished prioritizing the instance: result=" << success;
- if (!success)
- LOG(WARNING) << "Failed to prioritize ARC";
-}
-
-} // namespace
-
ArcBootPhaseMonitorBridge::ArcBootPhaseMonitorBridge(
ArcBridgeService* bridge_service)
: ArcService(bridge_service), binding_(this) {
@@ -48,11 +39,14 @@ void ArcBootPhaseMonitorBridge::OnInstanceClosed() {
void ArcBootPhaseMonitorBridge::OnBootCompleted() {
DCHECK(thread_checker_.CalledOnValidThread());
VLOG(2) << "OnBootCompleted";
+
chromeos::SessionManagerClient* session_manager_client =
chromeos::DBusThreadManager::Get()->GetSessionManagerClient();
- session_manager_client->PrioritizeArcInstance(
- base::Bind(PrioritizeArcInstanceCallback));
session_manager_client->EmitArcBooted();
+
+ // Start monitoring window activation changes to prioritize/throttle the
+ // container when needed.
+ throttle_ = base::MakeUnique<ArcInstanceThrottle>();
}
} // namespace arc

Powered by Google App Engine
This is Rietveld 408576698