ash: Drop the immediately following power button released induced forcing off request

ash/system/chromeos/power/tablet_power_button_controller_unittest.cc

Index: ash/system/chromeos/power/tablet_power_button_controller_unittest.cc
diff --git a/ash/system/chromeos/power/tablet_power_button_controller_unittest.cc b/ash/system/chromeos/power/tablet_power_button_controller_unittest.cc
index 2dcc5ece2e72ddadc54012374e8765736ae29afd..f337cbd2a51a7886119751690c2ed8fccbdb654a 100644
--- a/ash/system/chromeos/power/tablet_power_button_controller_unittest.cc
+++ b/ash/system/chromeos/power/tablet_power_button_controller_unittest.cc
@@ -439,5 +439,41 @@ TEST_F(TabletPowerButtonControllerTest,
   EXPECT_FALSE(GetBacklightsForcedOff());
 }
 
+// Tests that the very adjacent power button released for forcing off needs to

[Daniel Erat, 2017/01/30 17:55:00]

// Tests that repeated power button releases are ignored (crbug...).

?

[Qiang(Joe) Xu, 2017/01/30 19:35:44]

Done.

+// be dropped (crbug.com/675291).
+TEST_F(TabletPowerButtonControllerTest, DropVeryAdjacentForcingOff) {
+  // Advance a long duration from initialized last resume time in
+  // |tablet_controller_| to avoid cross interference.
+  tick_clock_->Advance(base::TimeDelta::FromMilliseconds(2000));
+
+  // Set backlights forced off for starting point.
+  PressPowerButton();
+  ReleasePowerButton();
+  power_manager_client_->SendBrightnessChanged(0, false);
+  EXPECT_TRUE(GetBacklightsForcedOff());
+
+  // Test that a pressing-releasing operation after a short duration, backlights
+  // forced off is stopped since we don't drop request for power button pressed.
+  tick_clock_->Advance(base::TimeDelta::FromMilliseconds(200));
+  power_manager_client_->SendPowerButtonEvent(true, tick_clock_->NowTicks());
+  power_manager_client_->SendBrightnessChanged(kNonZeroBrightness, false);
+  power_manager_client_->SendPowerButtonEvent(false, tick_clock_->NowTicks());
+  EXPECT_FALSE(GetBacklightsForcedOff());
+
+  // Test that after another short duration, backlights will not be forced off
+  // since this very adjacent forcing off request needs to be dropped.

[Daniel Erat, 2017/01/30 17:55:00]

hmm. "adjacent" usually refers more to physical distance than time distance, i
think. maybe something like "soon after" or "quickly following" or "immediately
following"?

[Qiang(Joe) Xu, 2017/01/30 19:35:44]

done by changed to "immediately following"

+  tick_clock_->Advance(base::TimeDelta::FromMilliseconds(200));
+  power_manager_client_->SendPowerButtonEvent(true, tick_clock_->NowTicks());
+  power_manager_client_->SendPowerButtonEvent(false, tick_clock_->NowTicks());
+  EXPECT_FALSE(GetBacklightsForcedOff());
+
+  // Test that after another long duration, backlights should be forced off.
+  tick_clock_->Advance(base::TimeDelta::FromMilliseconds(800));
+  power_manager_client_->SendPowerButtonEvent(true, tick_clock_->NowTicks());
+  power_manager_client_->SendPowerButtonEvent(false, tick_clock_->NowTicks());
+  power_manager_client_->SendBrightnessChanged(0, false);
+  EXPECT_TRUE(GetBacklightsForcedOff());
+}
+
 }  // namespace test
 }  // namespace ash