Simplify WindowManager::OnWmSetBounds

ui/aura/mus/window_manager_delegate.h

 // Copyright 2015 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 UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_
 #define UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 58 matching lines @@
 
 // Used by clients implementing a window manager.
 // TODO(sky): this should be called WindowManager, but that's rather confusing
 // currently.
 class AURA_EXPORT WindowManagerDelegate {
  public:
   // Called once to give the delegate access to functions only exposed to
   // the WindowManager.
   virtual void SetWindowManagerClient(WindowManagerClient* client) = 0;
 
-  // A client requested the bounds of |window| to change to |bounds|. Return
-  // true if the bounds are allowed to change. A return value of false
-  // indicates the change is not allowed.
-  // NOTE: This should not change the bounds of |window|. Instead return the
-  // bounds the window should be in |bounds|.
-  virtual bool OnWmSetBounds(Window* window, gfx::Rect* bounds) = 0;
+  // A client requested the bounds of |window| to change to |bounds|.
+  virtual void OnWmSetBounds(Window* window, const gfx::Rect& bounds) = 0;
 
   // A client requested the shared property named |name| to change to
   // |new_data|. Return true to allow the change to |new_data|, false
   // otherwise. If true is returned the property is set via
   // PropertyConverter::SetPropertyFromTransportValue().
   virtual bool OnWmSetProperty(
       Window* window,
       const std::string& name,
       std::unique_ptr<std::vector<uint8_t>>* new_data) = 0;
 
@@ skipping 88 matching lines @@
   // window.
   virtual void OnWmDeactivateWindow(Window* window) = 0;
 
  protected:
   virtual ~WindowManagerDelegate() {}
 };
 
 }  // namespace ui
 
 #endif  // UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_