Ports LockLayoutManager/LockWindowState to ash/common

ash/common/wm/lock_window_state.cc

Index: ash/common/wm/lock_window_state.cc
diff --git a/ash/wm/lock_window_state.cc b/ash/common/wm/lock_window_state.cc
similarity index 85%
rename from ash/wm/lock_window_state.cc
rename to ash/common/wm/lock_window_state.cc
index 18061e4e6f8dd303ad29aabf5f047a5369e6594e..e7d6ebe57a09274eecf23343d7b3c4c594bc0f03 100644
--- a/ash/wm/lock_window_state.cc
+++ b/ash/common/wm/lock_window_state.cc
@@ -2,11 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "ash/wm/lock_window_state.h"
+#include "ash/common/wm/lock_window_state.h"
 
 #include <utility>
 
-#include "ash/aura/wm_window_aura.h"
+#include "ash/common/wm/lock_layout_manager.h"
 #include "ash/common/wm/window_animation_types.h"
 #include "ash/common/wm/window_state.h"
 #include "ash/common/wm/window_state_delegate.h"
@@ -14,23 +14,15 @@
 #include "ash/common/wm/wm_event.h"
 #include "ash/common/wm/wm_screen_util.h"
 #include "ash/common/wm_shell.h"
-#include "ash/display/display_manager.h"
-#include "ash/shell.h"
-#include "ash/wm/lock_layout_manager.h"
-#include "ash/wm/window_animations.h"
-#include "ash/wm/window_state_aura.h"
-#include "ash/wm/window_util.h"
-#include "ui/aura/window.h"
-#include "ui/aura/window_delegate.h"
+#include "ash/common/wm_window.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/keyboard/keyboard_controller.h"
 #include "ui/keyboard/keyboard_util.h"
-#include "ui/wm/core/window_animations.h"
 
 namespace ash {
 
-LockWindowState::LockWindowState(aura::Window* window)
-    : current_state_type_(wm::GetWindowState(window)->GetStateType()) {}
+LockWindowState::LockWindowState(WmWindow* window)
+    : current_state_type_(window->GetWindowState()->GetStateType()) {}
 
 LockWindowState::~LockWindowState() {}
 
@@ -112,12 +104,12 @@ void LockWindowState::AttachState(wm::WindowState* window_state,
 void LockWindowState::DetachState(wm::WindowState* window_state) {}
 
 // static
-wm::WindowState* LockWindowState::SetLockWindowState(aura::Window* window) {
-  std::unique_ptr<wm::WindowState::State> lock_state(
-      new LockWindowState(window));
+wm::WindowState* LockWindowState::SetLockWindowState(WmWindow* window) {
+  std::unique_ptr<wm::WindowState::State> lock_state =
+      base::MakeUnique<LockWindowState>(window);
   std::unique_ptr<wm::WindowState::State> old_state(

[James Cook, 2016/09/20 18:22:47]

Aside: old_state is unused. Do you know if we have to do this to ignore the
return value?

-      wm::GetWindowState(window)->SetStateObject(std::move(lock_state)));
-  return wm::GetWindowState(window);
+      window->GetWindowState()->SetStateObject(std::move(lock_state)));
+  return window->GetWindowState();
 }
 
 void LockWindowState::UpdateWindow(wm::WindowState* window_state,
@@ -169,12 +161,6 @@ wm::WindowStateType LockWindowState::GetMaximizedOrCenteredWindowType(
                                      : wm::WINDOW_STATE_TYPE_NORMAL;
 }
 
-gfx::Rect GetBoundsForLockWindow(aura::Window* window) {

[James Cook, 2016/09/20 18:22:47]

Hooray for dead code removal!

-  if (WmShell::Get()->IsInUnifiedMode())
-    return WmShell::Get()->GetFirstDisplay().bounds();
-  return wm::GetDisplayBoundsInParent(WmWindowAura::Get(window));
-}
-
 void LockWindowState::UpdateBounds(wm::WindowState* window_state) {
   if (!window_state->IsMaximized() && !window_state->IsFullscreen())
     return;