diff options
-rw-r--r-- | DEPS | 2 | ||||
-rw-r--r-- | chrome/browser/chromeos/preferences.cc | 14 | ||||
-rw-r--r-- | chrome/browser/chromeos/preferences.h | 2 | ||||
-rw-r--r-- | chrome/common/pref_names.cc | 9 | ||||
-rw-r--r-- | chrome/common/pref_names.h | 1 | ||||
-rw-r--r-- | chromeos/dbus/power_policy_controller.cc | 12 | ||||
-rw-r--r-- | chromeos/dbus/power_policy_controller.h | 4 | ||||
-rw-r--r-- | chromeos/dbus/power_policy_controller_unittest.cc | 4 |
8 files changed, 29 insertions, 19 deletions
@@ -517,7 +517,7 @@ deps_os = { # For Linux and Chromium OS. "src/third_party/cros_system_api": Var("chromiumos_git") + "/platform/system_api.git" + - "@8d7776936afd26a16ba8b5103dc8bf14276ce6b5", + "@d2d441c18212d24fec220a24e81cf36957c70b14", # Note that this is different from Android's freetype repo. "src/third_party/freetype2/src": diff --git a/chrome/browser/chromeos/preferences.cc b/chrome/browser/chromeos/preferences.cc index da7dfcb..41b3e64 100644 --- a/chrome/browser/chromeos/preferences.cc +++ b/chrome/browser/chromeos/preferences.cc @@ -394,6 +394,10 @@ void Preferences::RegisterUserPrefs( 2.0, user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF); registry->RegisterDoublePref( + prefs::kPowerPresentationScreenDimDelayFactor, + 2.0, + user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF); + registry->RegisterDoublePref( prefs::kPowerUserActivityScreenDimDelayFactor, 2.0, user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF); @@ -529,8 +533,8 @@ void Preferences::InitUserPrefs(PrefServiceSyncable* prefs) { prefs::kPowerUseVideoActivity, prefs, callback); power_allow_screen_wake_locks_.Init( prefs::kPowerAllowScreenWakeLocks, prefs, callback); - power_presentation_idle_delay_factor_.Init( - prefs::kPowerPresentationIdleDelayFactor, prefs, callback); + power_presentation_screen_dim_delay_factor_.Init( + prefs::kPowerPresentationScreenDimDelayFactor, prefs, callback); power_user_activity_screen_dim_delay_factor_.Init( prefs::kPowerUserActivityScreenDimDelayFactor, prefs, callback); } @@ -856,7 +860,7 @@ void Preferences::NotifyPrefChanged(const std::string* pref_name) { *pref_name == prefs::kPowerUseAudioActivity || *pref_name == prefs::kPowerUseVideoActivity || *pref_name == prefs::kPowerAllowScreenWakeLocks || - *pref_name == prefs::kPowerPresentationIdleDelayFactor || + *pref_name == prefs::kPowerPresentationScreenDimDelayFactor || *pref_name == prefs::kPowerUserActivityScreenDimDelayFactor || *pref_name == prefs::kEnableScreenLock) { PowerPolicyController::PrefValues values; @@ -883,8 +887,8 @@ void Preferences::NotifyPrefChanged(const std::string* pref_name) { values.use_video_activity = power_use_video_activity_.GetValue(); values.allow_screen_wake_locks = power_allow_screen_wake_locks_.GetValue(); values.enable_screen_lock = enable_screen_lock_.GetValue(); - values.presentation_idle_delay_factor = - power_presentation_idle_delay_factor_.GetValue(); + values.presentation_screen_dim_delay_factor = + power_presentation_screen_dim_delay_factor_.GetValue(); values.user_activity_screen_dim_delay_factor = power_user_activity_screen_dim_delay_factor_.GetValue(); diff --git a/chrome/browser/chromeos/preferences.h b/chrome/browser/chromeos/preferences.h index 37be11d..e23c051 100644 --- a/chrome/browser/chromeos/preferences.h +++ b/chrome/browser/chromeos/preferences.h @@ -185,7 +185,7 @@ class Preferences : public PrefServiceSyncableObserver { BooleanPrefMember power_use_audio_activity_; BooleanPrefMember power_use_video_activity_; BooleanPrefMember power_allow_screen_wake_locks_; - DoublePrefMember power_presentation_idle_delay_factor_; + DoublePrefMember power_presentation_screen_dim_delay_factor_; DoublePrefMember power_user_activity_screen_dim_delay_factor_; DISALLOW_COPY_AND_ASSIGN(Preferences); diff --git a/chrome/common/pref_names.cc b/chrome/common/pref_names.cc index 97221a3..0d49f70 100644 --- a/chrome/common/pref_names.cc +++ b/chrome/common/pref_names.cc @@ -867,11 +867,16 @@ const char kPowerUseVideoActivity[] = "power.use_video_activity"; // screen-related power management (including locking)? const char kPowerAllowScreenWakeLocks[] = "power.allow_screen_wake_locks"; -// Amount by which the idle delay should be scaled while the system is in -// presentation mode. Values are limited to a minimum of 1.0. +// DEPRECATED: This is replaced by kPowerPresentationScreenDimDelayFactor +// and has no effect. const char kPowerPresentationIdleDelayFactor[] = "power.presentation_idle_delay_factor"; +// Amount by which the screen-dim delay should be scaled while the system +// is in presentation mode. Values are limited to a minimum of 1.0. +const char kPowerPresentationScreenDimDelayFactor[] = + "power.presentation_screen_dim_delay_factor"; + // Amount by which the screen-dim delay should be scaled when user activity is // observed while the screen is dimmed or soon after the screen has been turned // off. Values are limited to a minimum of 1.0. diff --git a/chrome/common/pref_names.h b/chrome/common/pref_names.h index 19f610a..b25e8d7 100644 --- a/chrome/common/pref_names.h +++ b/chrome/common/pref_names.h @@ -313,6 +313,7 @@ extern const char kPowerUseAudioActivity[]; extern const char kPowerUseVideoActivity[]; extern const char kPowerAllowScreenWakeLocks[]; extern const char kPowerPresentationIdleDelayFactor[]; +extern const char kPowerPresentationScreenDimDelayFactor[]; extern const char kPowerUserActivityScreenDimDelayFactor[]; extern const char kTermsOfServiceURL[]; extern const char kUsedPolicyCertificatesOnce[]; diff --git a/chromeos/dbus/power_policy_controller.cc b/chromeos/dbus/power_policy_controller.cc index 0485686..9ad9130 100644 --- a/chromeos/dbus/power_policy_controller.cc +++ b/chromeos/dbus/power_policy_controller.cc @@ -82,7 +82,7 @@ PowerPolicyController::PrefValues::PrefValues() use_video_activity(true), allow_screen_wake_locks(true), enable_screen_lock(false), - presentation_idle_delay_factor(2.0), + presentation_screen_dim_delay_factor(1.0), user_activity_screen_dim_delay_factor(1.0) {} // static @@ -101,9 +101,9 @@ std::string PowerPolicyController::GetPolicyDebugString( str += base::StringPrintf("use_audio=%d ", policy.use_audio_activity()); if (policy.has_use_video_activity()) str += base::StringPrintf("use_video=%d ", policy.use_audio_activity()); - if (policy.has_presentation_idle_delay_factor()) { - str += base::StringPrintf("presentation_idle_delay_factor=%f ", - policy.presentation_idle_delay_factor()); + if (policy.has_presentation_screen_dim_delay_factor()) { + str += base::StringPrintf("presentation_screen_dim_delay_factor=%f ", + policy.presentation_screen_dim_delay_factor()); } if (policy.has_user_activity_screen_dim_delay_factor()) { str += base::StringPrintf("user_activity_screen_dim_delay_factor=%f ", @@ -174,8 +174,8 @@ void PowerPolicyController::ApplyPrefs(const PrefValues& values) { prefs_policy_.set_lid_closed_action(GetProtoAction(values.lid_closed_action)); prefs_policy_.set_use_audio_activity(values.use_audio_activity); prefs_policy_.set_use_video_activity(values.use_video_activity); - prefs_policy_.set_presentation_idle_delay_factor( - values.presentation_idle_delay_factor); + prefs_policy_.set_presentation_screen_dim_delay_factor( + values.presentation_screen_dim_delay_factor); prefs_policy_.set_user_activity_screen_dim_delay_factor( values.user_activity_screen_dim_delay_factor); diff --git a/chromeos/dbus/power_policy_controller.h b/chromeos/dbus/power_policy_controller.h index ce5ed05..a2735a1 100644 --- a/chromeos/dbus/power_policy_controller.h +++ b/chromeos/dbus/power_policy_controller.h @@ -53,7 +53,7 @@ class CHROMEOS_EXPORT PowerPolicyController bool use_video_activity; bool allow_screen_wake_locks; bool enable_screen_lock; - double presentation_idle_delay_factor; + double presentation_screen_dim_delay_factor; double user_activity_screen_dim_delay_factor; }; @@ -65,7 +65,7 @@ class CHROMEOS_EXPORT PowerPolicyController // screen being locked. Used if the |enable_screen_lock| pref is set but // |*_screen_lock_delay_ms| are unset or set to higher values than what // this constant would imply. - const static int kScreenLockAfterOffDelayMs; + static const int kScreenLockAfterOffDelayMs; PowerPolicyController(DBusThreadManager* manager, PowerManagerClient* client); virtual ~PowerPolicyController(); diff --git a/chromeos/dbus/power_policy_controller_unittest.cc b/chromeos/dbus/power_policy_controller_unittest.cc index fe3601ef..b3b2564 100644 --- a/chromeos/dbus/power_policy_controller_unittest.cc +++ b/chromeos/dbus/power_policy_controller_unittest.cc @@ -53,7 +53,7 @@ TEST_F(PowerPolicyControllerTest, Prefs) { prefs.use_audio_activity = true; prefs.use_video_activity = true; prefs.enable_screen_lock = false; - prefs.presentation_idle_delay_factor = 3.0; + prefs.presentation_screen_dim_delay_factor = 3.0; prefs.user_activity_screen_dim_delay_factor = 2.0; policy_controller_->ApplyPrefs(prefs); @@ -74,7 +74,7 @@ TEST_F(PowerPolicyControllerTest, Prefs) { 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(3.0); + expected_policy.set_presentation_screen_dim_delay_factor(3.0); expected_policy.set_user_activity_screen_dim_delay_factor(2.0); expected_policy.set_reason("Prefs"); EXPECT_EQ(PowerPolicyController::GetPolicyDebugString(expected_policy), |