chromeos: Add power management policy prefs.

chrome/common/pref_names.cc

 // Copyright (c) 2012 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/common/pref_names.h"
 
 #include "base/basictypes.h"
 
 namespace prefs {
 
@@ skipping 810 matching lines @@
 // after login. When the browser restarts after a crash, the pref value is not
 // changed unless it appears corrupted (value unset, value lying in the future,
 // zero value).
 const char kSessionStartTime[] = "session.start_time";
 
 // Holds the maximum session time in milliseconds. If this pref is set, the
 // user is logged out when the maximum session time is reached. The user is
 // informed about the remaining time by a countdown timer shown in the ash
 // system tray.
 const char kSessionLengthLimit[] = "session.length_limit";
+
+// Inactivity time in milliseconds while the system is on AC power before
+// the screen should be dimmed, turned off, or locked, or before
+// kPowerIdleAction should be performed.  0 disables the delay (N/A for
+// kPowerAcIdleDelayMs); -1 instructs the power manager to use its default.

[bartfab (slow), 2013/02/04 10:04:35]

As commented in preferences.cc, there is no need for a magic -1.

+const char kPowerAcScreenDimDelayMs[] = "power.ac_screen_dim_delay_ms";
+const char kPowerAcScreenOffDelayMs[] = "power.ac_screen_off_delay_ms";
+const char kPowerAcScreenLockDelayMs[] = "power.ac_screen_lock_delay_ms";
+const char kPowerAcIdleDelayMs[] = "power.ac_idle_delay_ms";
+
+// Similar delays while the system is on battery power.
+const char kPowerBatteryScreenDimDelayMs[] =
+    "power.battery_screen_dim_delay_ms";
+const char kPowerBatteryScreenOffDelayMs[] =
+    "power.battery_screen_off_delay_ms";
+const char kPowerBatteryScreenLockDelayMs[] =
+    "power.battery_screen_lock_delay_ms";
+const char kPowerBatteryIdleDelayMs[] =
+    "power.battery_idle_delay_ms";
+
+// Action that should be performed when the idle delay is reached.
+// Values are from the chromeos::PowerPolicyController::OptionalAction enum.
+const char kPowerIdleAction[] = "power.idle_action";
+
+// Action that should be performed when the lid is closed.
+// Values are from the chromeos::PowerPolicyController::OptionalAction enum.
+const char kPowerLidClosedAction[] = "power.lid_closed_action";
+
+// Should audio and video activity be used to disable the above delays?
+// Values are from the chromeos::PowerPolicyController::OptionalBool enum.

[bartfab (slow), 2013/02/04 10:04:35]

As commented in preferences.cc, there is no need for an OptionalBool. A simple
bool will do.

+const char kPowerUseAudioActivity[] = "power.use_audio_activity";
+const char kPowerUseVideoActivity[] = "power.use_video_activity";
+
+// Amount by which the idle delay should be scaled while the system is in
+// presentation mode.  Values below 1.0 instruct the power manager to use
+// its default setting.

[bartfab (slow), 2013/02/04 10:04:35]

No need for a magic value. I would prefer if it just was "values below 1.0 are
clamped to 1.0".

+const char kPowerPresentationIdleDelayFactor[] =
+    "power.presentation_idle_delay_factor";
 #endif  // defined(OS_CHROMEOS)
 
 // The disabled messages in IPC logging.
 const char kIpcDisabledMessages[] = "ipc_log_disabled_messages";
 
 // A boolean pref set to true if a Home button to open the Home pages should be
 // visible on the toolbar.
 const char kShowHomeButton[] = "browser.show_home_button";
 
 // A string value which saves short list of recently user selected encodings
@@ skipping 1386 matching lines @@
 const char kLastPolicyStatisticsUpdate[] = "policy.last_statistics_update";
 
 #if defined(OS_CHROMEOS)
 // The RLZ brand code, if enabled.
 const char kRLZBrand[] = "rlz.brand";
 // Whether RLZ pings are disabled.
 const char kRLZDisabled[] = "rlz.disabled";
 #endif
 
 }  // namespace prefs