[Refactor] Move the logic to update bounds for show type from WorkspaceLayoutManager to DefaultState

ash/wm/wm_types.h

 // Copyright 2013 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.
 
 #ifndef ASH_WM_WM_TYPES_H_
 #define ASH_WM_WM_TYPES_H_
 
 #include "ash/ash_export.h"
 #include "ui/base/ui_base_types.h"
 
@@ skipping 25 matching lines @@
   SHOW_TYPE_RIGHT_SNAPPED,
 
   // A window is in this state when it is automatically placed and
   // sized by the window manager. (it's newly opened, or pushed to the side
   // due to new window, for example).
   SHOW_TYPE_AUTO_POSITIONED,
 };
 
 // Set of operations that can change the window's state.
 enum WMEvent {
+  // A window is requested to become a normal state. This usually invokes
+  // restore operation with a few exceptions.

[pkotwicz, 2014/02/12 22:27:27]

I do not think that the comment:  "A window is requested to become a normal
state" is a useful. Can you remove the sentence. (I think the second sentence is
still useful)

Can you also remove the comments for MAXIMIZE, MINIMIZE, FULLSCREEN, SNAP_LEFT,
SNAP_RIGHT events too (Keep the TODO)?

[oshima, 2014/02/13 14:52:34]

Removed individual comments, and added the overview of these events.

+  NORMAL,
+
+  // A window has been requested to become a maximized window.
+  MAXIMIZE,
+
+  // A window has been  requested to become a minimized window.
+  MINIMIZE,
+
+  // A window has been  requested to become a fullscreen window.
+  FULLSCREEN,
+
+  // A window has been  requested to become a left/right snapped window.
+  // TODO(oshima): Consolidate these two events.
+  SNAP_LEFT,
+  SNAP_RIGHT,
+
   // A user requested to toggle maximized state by double clicking window
   // header.
   TOGGLE_MAXIMIZE_CAPTION,
 
   // A user requested to toggle maximized state using shortcut.
   TOGGLE_MAXIMIZE,
 
   // A user requested to toggle vertical maximize by double clicking
   // top/bottom edge.
   TOGGLE_VERTICAL_MAXIMIZE,
 
   // A user requested to toggle horizontal maximize by double clicking
   // left/right edge.
   TOGGLE_HORIZONTAL_MAXIMIZE,
+
+  // A user requested to toggle fullscreen state.
+  TOGGLE_FULLSCREEN,
 };
 
 // Utility functions to convert WindowShowType <-> ui::WindowShowState.
 // Note: LEFT/RIGHT MAXIMIZED, AUTO_POSITIONED type will be lost when
 // converting to ui::WindowShowState.
 ASH_EXPORT WindowShowType ToWindowShowType(ui::WindowShowState state);
 ASH_EXPORT ui::WindowShowState ToWindowShowState(WindowShowType type);
 
 // Returns true if |type| is SHOW_TYPE_MAXIMIZED or SHOW_TYPE_FULLSCREEN.
 ASH_EXPORT bool IsMaximizedOrFullscreenWindowShowType(WindowShowType type);
 
 }  // namespace wm
 }  // namespace ash
 
 #endif  // ASH_WM_WM_TYPES_H_