Enabled/disable touch screen in TabletPowerButtonController

ash/system/chromeos/power/tablet_power_button_controller.cc

 // Copyright 2016 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 "ash/system/chromeos/power/tablet_power_button_controller.h"
 
 #include "ash/common/accessibility_delegate.h"
 #include "ash/common/session/session_state_delegate.h"
+#include "ash/common/shell_delegate.h"
 #include "ash/common/wm/maximize_mode/maximize_mode_controller.h"
 #include "ash/common/wm_shell.h"
 #include "ash/shell.h"
 #include "ash/wm/lock_state_controller.h"
 #include "base/time/default_tick_clock.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "ui/events/devices/input_device_manager.h"
 #include "ui/events/devices/stylus_state.h"
 #include "ui/events/event.h"
 
@@ skipping 70 matching lines @@
 }
 
 void TabletPowerButtonController::OnPowerButtonEvent(
     bool down,
     const base::TimeTicks& timestamp) {
   // When the system resumes in response to the power button being pressed,
   // Chrome receives powerd's SuspendDone signal and notification that the
   // backlight has been turned back on before seeing the power button events
   // that woke the system. Ignore events just after resuming to ensure that we
   // don't turn the screen off in response to the events.
+  //
+  // TODO(warx): pressing power button should also StartShutdownTimer() in this
+  // case. Reorganize the code to support that.
   if (timestamp - last_resume_time_ <=
       base::TimeDelta::FromMilliseconds(kIgnorePowerButtonAfterResumeMs)) {
     // If backlights are forced off, stop forcing off because resuming system
     // doesn't handle this.
     if (down && backlights_forced_off_)
-      SetBacklightsForcedOff(false);
+      SetDisplayForcedOff(false);
     return;
   }
 
   if (down) {
     screen_off_when_power_button_down_ = brightness_level_is_zero_;
-    SetBacklightsForcedOff(false);
+    SetDisplayForcedOff(false);
     StartShutdownTimer();
   } else {
     if (shutdown_timer_.IsRunning()) {
       shutdown_timer_.Stop();
       if (!screen_off_when_power_button_down_) {
-        SetBacklightsForcedOff(true);
+        SetDisplayForcedOff(true);
         LockScreenIfRequired();
       }
     }
     screen_off_when_power_button_down_ = false;
 
     // When power button is released, cancel shutdown animation whenever it is
     // still cancellable.
     if (controller_->CanCancelShutdownAnimation())
       controller_->CancelShutdownAnimation();
   }
@@ skipping 15 matching lines @@
   last_resume_time_ = tick_clock_->NowTicks();
 }
 
 void TabletPowerButtonController::OnKeyEvent(ui::KeyEvent* event) {
   // Ignore key events generated by the power button since power button activity
   // is already handled by OnPowerButtonEvent().
   if (event->key_code() == ui::VKEY_POWER)
     return;
 
   if (!IsTabletModeActive() && backlights_forced_off_)
-    SetBacklightsForcedOff(false);
+    SetDisplayForcedOff(false);
 }
 
 void TabletPowerButtonController::OnMouseEvent(ui::MouseEvent* event) {
   ui::EventPointerType pointer_type = event->pointer_details().pointer_type;
 
   if (pointer_type != ui::EventPointerType::POINTER_TYPE_MOUSE ||
       (event->flags() & ui::EF_IS_SYNTHESIZED)) {
     return;
   }
 
   if (!IsTabletModeActive() && backlights_forced_off_)
-    SetBacklightsForcedOff(false);
+    SetDisplayForcedOff(false);
 }
 
 void TabletPowerButtonController::OnStylusStateChanged(ui::StylusState state) {
   if (IsTabletModeSupported() && state == ui::StylusState::REMOVED &&
       backlights_forced_off_) {
-    SetBacklightsForcedOff(false);
+    SetDisplayForcedOff(false);
   }
 }
 
 void TabletPowerButtonController::SetTickClockForTesting(
     std::unique_ptr<base::TickClock> tick_clock) {
   DCHECK(tick_clock);
   tick_clock_ = std::move(tick_clock);
 }
 
-void TabletPowerButtonController::SetBacklightsForcedOff(bool forced_off) {
+void TabletPowerButtonController::SetDisplayForcedOff(bool forced_off) {
   if (backlights_forced_off_ == forced_off)
     return;
 
+  // Set the display and keyboard backlights (if present) to |forced_off|.
   chromeos::DBusThreadManager::Get()
       ->GetPowerManagerClient()
       ->SetBacklightsForcedOff(forced_off);
   backlights_forced_off_ = forced_off;
 
+  bool touch_screen_enabled = !forced_off;

[Daniel Erat, 2016/12/02 21:48:56]

nit: this is only used in one place, so it'd probably be cleaner to inline it in
the SetTouchscreenEnabledInPrefs call

[Qiang(Joe) Xu, 2016/12/02 23:55:10]

Done.

+  ShellDelegate* delegate = WmShell::Get()->delegate();
+  delegate->SetTouchscreenEnabledInPrefs(touch_screen_enabled,
+                                         true /* is_local */);
+  delegate->UpdateTouchscreenStatusFromPrefs();
+
   // Send an a11y alert.
   WmShell::Get()->accessibility_delegate()->TriggerAccessibilityAlert(
       forced_off ? A11Y_ALERT_SCREEN_OFF : A11Y_ALERT_SCREEN_ON);
 }
 
 void TabletPowerButtonController::GetInitialBacklightsForcedOff() {
   chromeos::DBusThreadManager::Get()
       ->GetPowerManagerClient()
       ->GetBacklightsForcedOff(base::Bind(
           &TabletPowerButtonController::OnGotInitialBacklightsForcedOff,
@@ skipping 20 matching lines @@
       WmShell::Get()->GetSessionStateDelegate();
   if (session_state_delegate->ShouldLockScreenAutomatically() &&
       session_state_delegate->CanLockScreen() &&
       !session_state_delegate->IsUserSessionBlocked() &&
       !controller_->LockRequested()) {
     session_state_delegate->LockScreen();
   }
 }
 
 }  // namespace ash