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

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

Issue 2577103002: arc: Rename methods in ArcSessionManager::Observer. (Closed)
Patch Set: OnArcShutdownBridge -> OnArcBridgeShutdown Created 4 years 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_list_prefs.cc
diff --git a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc b/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
index c7c8218560a6618a85f2ccb7e9790778e84889a1..56017efe935663165d4933c59895af8208010984 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
+++ b/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
@@ -269,7 +269,7 @@ void ArcAppListPrefs::StartPrefs() {
if (arc_session_manager->state() !=
arc::ArcSessionManager::State::NOT_INITIALIZED)
- OnOptInEnabled(arc_session_manager->IsArcEnabled());
+ OnArcOptInChanged(arc_session_manager->IsArcEnabled());
arc_session_manager->AddObserver(this);
app_instance_holder_->AddObserver(this);
@@ -593,7 +593,7 @@ void ArcAppListPrefs::RemoveAllApps() {
DCHECK(ready_apps_.empty());
}
-void ArcAppListPrefs::OnOptInEnabled(bool enabled) {
+void ArcAppListPrefs::OnArcOptInChanged(bool enabled) {
UpdateDefaultAppsHiddenState();
if (enabled)

Powered by Google App Engine
This is Rietveld 408576698