chromeos: Move default power management policy into Chrome.

chromeos/dbus/power_policy_controller_unittest.cc

Index: chromeos/dbus/power_policy_controller_unittest.cc
diff --git a/chromeos/dbus/power_policy_controller_unittest.cc b/chromeos/dbus/power_policy_controller_unittest.cc
index 8d9fb3cd7d89d32f9c947c096315e7993df18a0b..5cee3c58349ee43eb099436e6c68f136c9c86f6f 100644
--- a/chromeos/dbus/power_policy_controller_unittest.cc
+++ b/chromeos/dbus/power_policy_controller_unittest.cc
@@ -26,12 +26,7 @@ class PowerPolicyControllerTest : public testing::Test {
     power_client_ = dbus_manager_->mock_power_manager_client();
     EXPECT_CALL(*power_client_, SetPolicy(_))
         .WillRepeatedly(SaveArg<0>(&last_policy_));
-
     policy_controller_ = dbus_manager_->GetPowerPolicyController();
-
-    // TODO(derat): Write what looks like it will be a ridiculously large
-    // amount of code to register prefs so that UpdatePolicyFromPrefs() can
-    // be tested.
   }
 
   virtual void TearDown() OVERRIDE {
@@ -46,6 +41,76 @@ class PowerPolicyControllerTest : public testing::Test {
   power_manager::PowerManagementPolicy last_policy_;
 };
 
+TEST_F(PowerPolicyControllerTest, Prefs) {
+  PowerPolicyController::PrefValues prefs;
+  prefs.ac_screen_dim_delay_ms = 600000;
+  prefs.ac_screen_off_delay_ms = 660000;
+  prefs.ac_idle_delay_ms = 720000;
+  prefs.battery_screen_dim_delay_ms = 300000;
+  prefs.battery_screen_off_delay_ms = 360000;
+  prefs.battery_idle_delay_ms = 420000;
+  prefs.idle_action = PowerPolicyController::ACTION_SUSPEND;
+  prefs.lid_closed_action = PowerPolicyController::ACTION_SHUT_DOWN;
+  prefs.use_audio_activity = true;
+  prefs.use_video_activity = true;
+  prefs.enable_screen_lock = false;
+  prefs.presentation_idle_delay_factor = 2.0;
+  policy_controller_->ApplyPrefs(prefs);
+
+  power_manager::PowerManagementPolicy expected_policy;
+  expected_policy.mutable_ac_delays()->set_screen_dim_ms(600000);
+  expected_policy.mutable_ac_delays()->set_screen_off_ms(660000);
+  expected_policy.mutable_ac_delays()->set_screen_lock_ms(-1);
+  expected_policy.mutable_ac_delays()->set_idle_warning_ms(-1);
+  expected_policy.mutable_ac_delays()->set_idle_ms(720000);
+  expected_policy.mutable_battery_delays()->set_screen_dim_ms(300000);
+  expected_policy.mutable_battery_delays()->set_screen_off_ms(360000);
+  expected_policy.mutable_battery_delays()->set_screen_lock_ms(-1);
+  expected_policy.mutable_battery_delays()->set_idle_warning_ms(-1);
+  expected_policy.mutable_battery_delays()->set_idle_ms(420000);
+  expected_policy.set_idle_action(
+      power_manager::PowerManagementPolicy_Action_SUSPEND);
+  expected_policy.set_lid_closed_action(
+      power_manager::PowerManagementPolicy_Action_SHUT_DOWN);
+  expected_policy.set_use_audio_activity(true);
+  expected_policy.set_use_video_activity(true);
+  expected_policy.set_presentation_idle_delay_factor(2.0);
+  expected_policy.set_reason("Prefs");
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
+
+  // Change some prefs and check that an updated policy is sent.
+  prefs.ac_idle_warning_delay_ms = 700000;
+  prefs.battery_idle_warning_delay_ms = 400000;
+  prefs.lid_closed_action = PowerPolicyController::ACTION_SUSPEND;
+  policy_controller_->ApplyPrefs(prefs);
+  expected_policy.mutable_ac_delays()->set_idle_warning_ms(700000);
+  expected_policy.mutable_battery_delays()->set_idle_warning_ms(400000);
+  expected_policy.set_lid_closed_action(
+      power_manager::PowerManagementPolicy_Action_SUSPEND);
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
+
+  // The enable-screen-lock pref should force the screen-lock delays to
+  // match the screen-off delays.
+  prefs.enable_screen_lock = true;
+  policy_controller_->ApplyPrefs(prefs);
+  expected_policy.mutable_ac_delays()->set_screen_lock_ms(660000);
+  expected_policy.mutable_battery_delays()->set_screen_lock_ms(360000);
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
+
+  // If the screen-lock-delay prefs are set to lower values than the
+  // screen-off delays, the lock prefs should take precedence.
+  prefs.ac_screen_lock_delay_ms = 70000;
+  prefs.battery_screen_lock_delay_ms = 60000;
+  policy_controller_->ApplyPrefs(prefs);
+  expected_policy.mutable_ac_delays()->set_screen_lock_ms(70000);
+  expected_policy.mutable_battery_delays()->set_screen_lock_ms(60000);
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
+}
+
 TEST_F(PowerPolicyControllerTest, Blocks) {
   const char kSuspendBlockReason[] = "suspend";
   const int suspend_id =
@@ -54,8 +119,8 @@ TEST_F(PowerPolicyControllerTest, Blocks) {
   expected_policy.set_idle_action(
       power_manager::PowerManagementPolicy_Action_DO_NOTHING);
   expected_policy.set_reason(kSuspendBlockReason);
-  EXPECT_EQ(expected_policy.SerializeAsString(),
-            last_policy_.SerializeAsString());
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
 
   const char kScreenBlockReason[] = "screen";
   const int screen_id = policy_controller_->AddScreenBlock(kScreenBlockReason);
@@ -65,18 +130,18 @@ TEST_F(PowerPolicyControllerTest, Blocks) {
   expected_policy.mutable_battery_delays()->set_screen_off_ms(0);
   expected_policy.set_reason(
       std::string(kScreenBlockReason) + ", " + kSuspendBlockReason);
-  EXPECT_EQ(expected_policy.SerializeAsString(),
-            last_policy_.SerializeAsString());
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
 
   policy_controller_->RemoveBlock(suspend_id);
   expected_policy.set_reason(kScreenBlockReason);
-  EXPECT_EQ(expected_policy.SerializeAsString(),
-            last_policy_.SerializeAsString());
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
 
   policy_controller_->RemoveBlock(screen_id);
   expected_policy.Clear();
-  EXPECT_EQ(expected_policy.SerializeAsString(),
-            last_policy_.SerializeAsString());
+  EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy),
+            PowerPolicyController::GetPolicyDebugString(last_policy_));
 }
 
 }  // namespace chromeos