Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1335)

Unified Diff: ash/system/chromeos/power/tablet_power_button_controller.cc

Issue 2640333002: Revert "ash: Add one second grace period For LockScreenIfRequired" (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ash/system/chromeos/power/tablet_power_button_controller.cc
diff --git a/ash/system/chromeos/power/tablet_power_button_controller.cc b/ash/system/chromeos/power/tablet_power_button_controller.cc
index aba3b6cca4f4df6d1cdc6e0079d7c74d8dc6deec..f34dc3ee71973f99955d421e89d595d1b59b9850 100644
--- a/ash/system/chromeos/power/tablet_power_button_controller.cc
+++ b/ash/system/chromeos/power/tablet_power_button_controller.cc
@@ -25,9 +25,6 @@ namespace {
// animation when in tablet mode.
constexpr int kShutdownTimeoutMs = 500;
-// Amount of time to delay locking screen after display is forced off.
-constexpr int kLockScreenTimeoutMs = 1000;
-
// Amount of time since last SuspendDone() that power button event needs to be
// ignored.
constexpr int kIgnorePowerButtonAfterResumeMs = 2000;
@@ -67,16 +64,6 @@ void TabletPowerButtonController::TestApi::TriggerShutdownTimeout() {
controller_->shutdown_timer_.Stop();
}
-bool TabletPowerButtonController::TestApi::LockScreenTimerIsRunning() const {
- return controller_->lock_screen_timer_.IsRunning();
-}
-
-void TabletPowerButtonController::TestApi::TriggerLockScreenTimeout() {
- DCHECK(LockScreenTimerIsRunning());
- controller_->OnLockScreenTimeout();
- controller_->lock_screen_timer_.Stop();
-}
-
TabletPowerButtonController::TabletPowerButtonController(
LockStateController* controller)
: tick_clock_(new base::DefaultTickClock()),
@@ -121,7 +108,6 @@ void TabletPowerButtonController::OnPowerButtonEvent(
}
screen_off_when_power_button_down_ = brightness_level_is_zero_;
SetDisplayForcedOff(false);
- lock_screen_timer_.Stop();
StartShutdownTimer();
} else {
if (shutdown_timer_.IsRunning()) {
@@ -173,27 +159,22 @@ void TabletPowerButtonController::OnKeyEvent(ui::KeyEvent* event) {
if (event->key_code() == ui::VKEY_POWER)
return;
- if (!IsTabletModeActive() && backlights_forced_off_) {
+ if (!IsTabletModeActive() && backlights_forced_off_)
SetDisplayForcedOff(false);
- lock_screen_timer_.Stop();
- }
}
void TabletPowerButtonController::OnMouseEvent(ui::MouseEvent* event) {
if (event->flags() & ui::EF_IS_SYNTHESIZED)
return;
- if (!IsTabletModeActive() && backlights_forced_off_) {
+ if (!IsTabletModeActive() && backlights_forced_off_)
SetDisplayForcedOff(false);
- lock_screen_timer_.Stop();
- }
}
void TabletPowerButtonController::OnStylusStateChanged(ui::StylusState state) {
if (IsTabletModeSupported() && state == ui::StylusState::REMOVED &&
backlights_forced_off_) {
SetDisplayForcedOff(false);
- lock_screen_timer_.Stop();
}
}
@@ -253,14 +234,8 @@ void TabletPowerButtonController::LockScreenIfRequired() {
session_state_delegate->CanLockScreen() &&
!session_state_delegate->IsUserSessionBlocked() &&
!controller_->LockRequested()) {
- lock_screen_timer_.Start(
- FROM_HERE, base::TimeDelta::FromMilliseconds(kLockScreenTimeoutMs),
- this, &TabletPowerButtonController::OnLockScreenTimeout);
+ session_state_delegate->LockScreen();
}
}
-void TabletPowerButtonController::OnLockScreenTimeout() {
- WmShell::Get()->GetSessionStateDelegate()->LockScreen();
-}
-
} // namespace ash

Powered by Google App Engine
This is Rietveld 408576698