Update bool WindowManager::OnWmSetBounds() to match with its desirable behavior.

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 65 matching lines @@
  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;
+  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 51 matching lines @@
       const gfx::Insets& insets,
       const std::vector<gfx::Rect>& additional_client_areas) = 0;
 
  protected:
   virtual ~WindowManagerDelegate() {}
 };
 
 }  // namespace ui
 
 #endif  // UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_