chromeos: Add power management policy prefs.

chrome/browser/chromeos/preferences.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_PREFERENCES_H_
 #define CHROME_BROWSER_CHROMEOS_PREFERENCES_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 78 matching lines @@
 
   // Updates the initial key repeat delay and key repeat interval following
   // current prefs values. We set the delay and interval at once since an
   // underlying XKB API requires it.
   void UpdateAutoRepeatRate();
 
   // Force natural scroll to on if --enable-natural-scroll-default is specified
   // on the cmd line.
   void ForceNaturalScrollDefault();
 
+  // Updates the power management policy to reflect the current values of
+  // |power_*_pref_|.
+  void UpdatePowerManagementPolicy();

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

It's a bit unfortunate that this is called "policy" as Chrome has a notion of
"policy" and it means something different.

Could you disambiguate in the comment a bit by using e.g. "power manager policy"
or "powerd policy"?

[Daniel Erat, 2013/02/04 13:35:45]

Not sure what you mean here -- this is already called
"UpdatePowerManagementPolicy()".  Do you mean you want it to be
"UpdatePowerManagerPolicy()" instead?

[bartfab (slow), 2013/02/04 14:43:52]

Yes, I was thinking of PowerManagerPolicy. That way, it should be a tad clearer
that this is not "policy" in the Chrome sense but policy in the sense of some
other entity, the power manager.

+
   // PrefServiceSyncableObserver implementation.
   virtual void OnIsSyncingChanged() OVERRIDE;
 
   PrefServiceSyncable* prefs_;
 
   input_method::InputMethodManager* input_method_manager_;
 
   BooleanPrefMember tap_to_click_enabled_;
   BooleanPrefMember tap_dragging_enabled_;
   BooleanPrefMember three_finger_click_enabled_;
@@ skipping 43 matching lines @@
   IntegerPrefMember mozc_integer_prefs_[
       language_prefs::kNumMozcIntegerPrefs];
   BooleanPrefMember xkb_auto_repeat_enabled_;
   IntegerPrefMember xkb_auto_repeat_delay_pref_;
   IntegerPrefMember xkb_auto_repeat_interval_pref_;
 
   BooleanPrefMember enable_screen_lock_;
 
   BooleanPrefMember enable_drm_;
 
+  // Power-management-related preferences.
+  IntegerPrefMember power_ac_screen_dim_delay_ms_pref_;
+  IntegerPrefMember power_ac_screen_off_delay_ms_pref_;
+  IntegerPrefMember power_ac_screen_lock_delay_ms_pref_;
+  IntegerPrefMember power_ac_idle_delay_ms_pref_;
+  IntegerPrefMember power_battery_screen_dim_delay_ms_pref_;
+  IntegerPrefMember power_battery_screen_off_delay_ms_pref_;
+  IntegerPrefMember power_battery_screen_lock_delay_ms_pref_;
+  IntegerPrefMember power_battery_idle_delay_ms_pref_;
+  IntegerPrefMember power_idle_action_pref_;
+  IntegerPrefMember power_lid_closed_action_pref_;
+  IntegerPrefMember power_use_audio_activity_pref_;

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

See my comment in the preferences.cc file - this can be expressed as a simple
bool.

+  IntegerPrefMember power_use_video_activity_pref_;

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

See my comment in the preferences.cc file - this can be expressed as a simple
bool.

+  DoublePrefMember power_presentation_idle_delay_factor_pref_;
+
   DISALLOW_COPY_AND_ASSIGN(Preferences);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_PREFERENCES_H_