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

Unified Diff: ash/accelerators/accelerator_controller_delegate_aura.cc

Issue 2720913003: Calls display_configuration_controller to rotate screen in accelerator_controller_delegate_aura (Closed)
Patch Set: Created 3 years, 10 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/accelerators/accelerator_controller_delegate_aura.cc
diff --git a/ash/accelerators/accelerator_controller_delegate_aura.cc b/ash/accelerators/accelerator_controller_delegate_aura.cc
index 16313bd039f056315c2d3d55a13655e0ca14c757..a81e19bfac10e52cf0e764c482eafd195fe2a92f 100644
--- a/ash/accelerators/accelerator_controller_delegate_aura.cc
+++ b/ash/accelerators/accelerator_controller_delegate_aura.cc
@@ -166,9 +166,9 @@ void HandleRotateScreen() {
display::Screen::GetScreen()->GetDisplayNearestPoint(point);
const display::ManagedDisplayInfo& display_info =
Shell::GetInstance()->display_manager()->GetDisplayInfo(display.id());
- ScreenRotationAnimator(display.id())
- .Rotate(GetNextRotation(display_info.GetActiveRotation()),
- display::Display::ROTATION_SOURCE_USER);
+ Shell::GetInstance()->display_configuration_controller()->SetDisplayRotation(
bruthig 2017/02/28 19:11:46 Do any SetDisplayRotation() call sites even pass f
wutao 2017/02/28 20:29:09 All 3 call sites pass true, therefore remove the p
+ display.id(), GetNextRotation(display_info.GetActiveRotation()),
+ display::Display::ROTATION_SOURCE_USER, true /*user_action*/);
}
// Rotate the active window.

Powered by Google App Engine
This is Rietveld 408576698