Add policies to control power management on the Chrome OS login screen

chrome/browser/chromeos/power/power_prefs.cc

+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/chromeos/power/power_prefs.h"
+
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/callback.h"
+#include "base/logging.h"
+#include "base/prefs/pref_change_registrar.h"
+#include "base/prefs/pref_service.h"
+#include "chrome/browser/chromeos/profiles/profile_helper.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/common/chrome_notification_types.h"
+#include "chrome/common/pref_names.h"
+#include "chromeos/dbus/power_policy_controller.h"
+#include "components/user_prefs/pref_registry_syncable.h"
+#include "content/public/browser/notification_details.h"
+#include "content/public/browser/notification_service.h"
+#include "content/public/browser/notification_source.h"
+
+namespace chromeos {
+
+PowerPrefs::PowerPrefs(PowerPolicyController* power_policy_controller)
+    : power_policy_controller_(power_policy_controller),
+      profile_(NULL) {
+  notification_registrar_.Add(this,
+                              chrome::NOTIFICATION_LOGIN_WEBUI_VISIBLE,
+                              content::NotificationService::AllSources());
+  notification_registrar_.Add(this,
+                              chrome::NOTIFICATION_SESSION_STARTED,
+                              content::NotificationService::AllSources());
+  notification_registrar_.Add(this,
+                              chrome::NOTIFICATION_PROFILE_DESTROYED,
+                              content::NotificationService::AllSources());
+}
+
+PowerPrefs::~PowerPrefs() {
+}
+
+// static
+void PowerPrefs::RegisterUserPrefs(user_prefs::PrefRegistrySyncable* registry,
+                                   bool for_signin_profile) {
+  registry->RegisterIntegerPref(
+      prefs::kPowerAcScreenDimDelayMs,
+      420000,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerAcScreenOffDelayMs,
+      480000,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerAcScreenLockDelayMs,
+      0,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerAcIdleWarningDelayMs,
+      0,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerAcIdleDelayMs,
+      1800000,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerBatteryScreenDimDelayMs,
+      300000,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerBatteryScreenOffDelayMs,
+      360000,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerBatteryScreenLockDelayMs,
+      0,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerBatteryIdleWarningDelayMs,
+      0,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerBatteryIdleDelayMs,
+      600000,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerAcIdleAction,
+      PowerPolicyController::ACTION_SUSPEND,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerBatteryIdleAction,
+      for_signin_profile ? PowerPolicyController::ACTION_SHUT_DOWN :

[Daniel Erat, 2013/06/28 17:05:09]

just to double-check: for_signin_profile will be false in guest mode, right?

[bartfab (slow), 2013/07/01 12:32:59]

Yes. The signin profile is a special profile used by the login screen / OOBE
mechanism only. The guest profile and all user profiles are entirely separate
from it.

+                           PowerPolicyController::ACTION_SUSPEND,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kPowerLidClosedAction,
+      for_signin_profile ? PowerPolicyController::ACTION_SHUT_DOWN :
+                           PowerPolicyController::ACTION_SUSPEND,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kPowerUseAudioActivity,
+      true,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kPowerUseVideoActivity,
+      true,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kPowerAllowScreenWakeLocks,
+      true,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kEnableScreenLock,

[Daniel Erat, 2013/06/28 17:05:09]

i'm not entirely convinced that this should be considered a
power-management-related pref (even though it's used by PowerPolicyController).
do you feel strongly one way or the other about it?

[bartfab (slow), 2013/07/01 12:32:59]

I do not feel strongly either way but considering the fact that the screen lock
pref is only actually read and used by the power management code, I think it
belongs here more than anywhere else.

[Daniel Erat, 2013/07/01 14:29:05]

okay, fine by me

+      false,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+  registry->RegisterDoublePref(
+      prefs::kPowerPresentationScreenDimDelayFactor,
+      2.0,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+  registry->RegisterDoublePref(
+      prefs::kPowerUserActivityScreenDimDelayFactor,
+      2.0,
+      user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+}
+
+void PowerPrefs::Observe(int type,
+                         const content::NotificationSource& source,
+                         const content::NotificationDetails& details) {
+  switch (type) {
+    case chrome::NOTIFICATION_LOGIN_WEBUI_VISIBLE:
+      // Update |profile_| when entering the login screen.
+      SetProfile(ProfileHelper::GetSigninProfile());
+      break;
+    case chrome::NOTIFICATION_SESSION_STARTED:
+      // Update |profile_| when entering a session.
+      SetProfile(ProfileManager::GetDefaultProfile());
+      break;
+    case chrome::NOTIFICATION_PROFILE_DESTROYED: {
+      // Update |profile_| when exiting a session or shutting down.
+      Profile* profile = content::Source<Profile>(source).ptr();
+      if (profile_ == profile)
+        SetProfile(NULL);
+      break;
+    }

[Daniel Erat, 2013/06/28 17:05:09]

do you need a default: NOTREACHED() here to keep clang from complaining?

[bartfab (slow), 2013/07/01 12:32:59]

I had not tried to compile this with clang yet. Makes sense to add a default:
NOTREACHED() to catch errors either way. Done.

+  }
+}
+
+void PowerPrefs::UpdatePowerPolicyFromPrefs() {
+  if (!pref_change_registrar_ || !pref_change_registrar_->prefs()) {
+    NOTREACHED();
+    return;
+  }
+
+  const PrefService* prefs = pref_change_registrar_->prefs();
+  PowerPolicyController::PrefValues values;
+  values.ac_screen_dim_delay_ms =
+      prefs->GetInteger(prefs::kPowerAcScreenDimDelayMs);
+  values.ac_screen_off_delay_ms =
+      prefs->GetInteger(prefs::kPowerAcScreenOffDelayMs);
+  values.ac_screen_lock_delay_ms =
+      prefs->GetInteger(prefs::kPowerAcScreenLockDelayMs);
+  values.ac_idle_warning_delay_ms =
+      prefs->GetInteger(prefs::kPowerAcIdleWarningDelayMs);
+  values.ac_idle_delay_ms =
+      prefs->GetInteger(prefs::kPowerAcIdleDelayMs);
+  values.battery_screen_dim_delay_ms =
+      prefs->GetInteger(prefs::kPowerBatteryScreenDimDelayMs);
+  values.battery_screen_off_delay_ms =
+      prefs->GetInteger(prefs::kPowerBatteryScreenOffDelayMs);
+  values.battery_screen_lock_delay_ms =
+      prefs->GetInteger(prefs::kPowerBatteryScreenLockDelayMs);
+  values.battery_idle_warning_delay_ms =
+      prefs->GetInteger(prefs::kPowerBatteryIdleWarningDelayMs);
+  values.battery_idle_delay_ms =
+      prefs->GetInteger(prefs::kPowerBatteryIdleDelayMs);
+  values.ac_idle_action = static_cast<PowerPolicyController::Action>(
+      prefs->GetInteger(prefs::kPowerAcIdleAction));
+  values.battery_idle_action = static_cast<PowerPolicyController::Action>(
+      prefs->GetInteger(prefs::kPowerBatteryIdleAction));
+  values.lid_closed_action = static_cast<PowerPolicyController::Action>(
+      prefs->GetInteger(prefs::kPowerLidClosedAction));
+  values.use_audio_activity =
+      prefs->GetBoolean(prefs::kPowerUseAudioActivity);
+  values.use_video_activity =
+      prefs->GetBoolean(prefs::kPowerUseVideoActivity);
+  values.allow_screen_wake_locks =
+      prefs->GetBoolean(prefs::kPowerAllowScreenWakeLocks);
+  values.enable_screen_lock =
+      prefs->GetBoolean(prefs::kEnableScreenLock);
+  values.presentation_screen_dim_delay_factor =
+      prefs->GetDouble(prefs::kPowerPresentationScreenDimDelayFactor);
+  values.user_activity_screen_dim_delay_factor =
+      prefs->GetDouble(prefs::kPowerUserActivityScreenDimDelayFactor);
+  power_policy_controller_->ApplyPrefs(values);
+}
+
+void PowerPrefs::SetProfile(Profile* profile) {
+  profile_ = profile;
+  pref_change_registrar_.reset();
+
+  if (!profile) {
+    power_policy_controller_->ClearPrefs();
+    return;
+  }
+
+  base::Closure update_callback(base::Bind(
+      &PowerPrefs::UpdatePowerPolicyFromPrefs,
+      base::Unretained(this)));
+  pref_change_registrar_.reset(new PrefChangeRegistrar);
+  pref_change_registrar_->Init(profile->GetPrefs());
+  pref_change_registrar_->Add(prefs::kPowerAcScreenDimDelayMs, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerAcScreenOffDelayMs, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerAcScreenLockDelayMs,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kPowerAcIdleWarningDelayMs,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kPowerAcIdleDelayMs, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerBatteryScreenDimDelayMs,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kPowerBatteryScreenOffDelayMs,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kPowerBatteryScreenLockDelayMs,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kPowerBatteryIdleWarningDelayMs,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kPowerBatteryIdleDelayMs, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerAcIdleAction, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerBatteryIdleAction, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerLidClosedAction, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerUseAudioActivity, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerUseVideoActivity, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerAllowScreenWakeLocks,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kEnableScreenLock, update_callback);
+  pref_change_registrar_->Add(prefs::kPowerPresentationScreenDimDelayFactor,
+                              update_callback);
+  pref_change_registrar_->Add(prefs::kPowerUserActivityScreenDimDelayFactor,
+                              update_callback);
+
+  UpdatePowerPolicyFromPrefs();
+}
+
+}  // namespace chromeos