ash: Do not constrain window bounds if requested

ash/common/wm/maximize_mode/maximize_mode_window_state.cc

 // Copyright 2014 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/common/wm/maximize_mode/maximize_mode_window_state.h"
 
 #include <utility>
 
 #include "ash/common/wm/maximize_mode/maximize_mode_window_manager.h"
 #include "ash/common/wm/window_animation_types.h"
@@ skipping 138 matching lines @@
     case wm::WM_EVENT_MAXIMIZE:
       UpdateWindow(window_state, GetMaximizedOrCenteredWindowType(window_state),
                    true);
       return;
     case wm::WM_EVENT_MINIMIZE:
       UpdateWindow(window_state, wm::WINDOW_STATE_TYPE_MINIMIZED, true);
       return;
     case wm::WM_EVENT_SHOW_INACTIVE:
       return;
     case wm::WM_EVENT_SET_BOUNDS:
-      if (window_state->allow_set_bounds_in_maximized()) {
-        window_state->SetBoundsConstrained(
+      if (window_state->allow_set_bounds_direct()) {

[oshima, 2017/04/12 22:27:24]

this is slate code. can you just remove this?

[Dominik Laskowski, 2017/04/12 22:45:40]

Done.

+        window_state->SetBoundsDirect(
             static_cast<const wm::SetBoundsEvent*>(event)->requested_bounds());
       } else if (current_state_type_ == wm::WINDOW_STATE_TYPE_MAXIMIZED) {
         // Having a maximized window, it could have been created with an empty
         // size and the caller should get his size upon leaving the maximized
         // mode. As such we set the restore bounds to the requested bounds.
         gfx::Rect bounds_in_parent =
             (static_cast<const wm::SetBoundsEvent*>(event))->requested_bounds();
         if (!bounds_in_parent.IsEmpty())
           window_state->SetRestoreBoundsInParent(bounds_in_parent);
       } else if (current_state_type_ != wm::WINDOW_STATE_TYPE_MINIMIZED &&
@@ skipping 147 matching lines @@
       // avoid flashing.
       if (window_state->IsMaximized())
         window_state->SetBoundsDirectCrossFade(bounds_in_parent);
       else
         window_state->SetBoundsDirectAnimated(bounds_in_parent);
     }
   }
 }
 
 }  // namespace ash