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

ash/mus/window_manager.cc

 // 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.
 
 #include "ash/mus/window_manager.h"
 
 #include <stdint.h>
 
 #include <utility>
 
@@ skipping 353 matching lines @@
 }
 
 aura::PropertyConverter* WindowManager::GetPropertyConverter() {
   return property_converter_.get();
 }
 
 void WindowManager::SetWindowManagerClient(aura::WindowManagerClient* client) {
   window_manager_client_ = client;
 }
 
-bool WindowManager::OnWmSetBounds(aura::Window* window, gfx::Rect* bounds) {
-  // TODO(sky): this indirectly sets bounds, which is against what
-  // OnWmSetBounds() recommends doing. Remove that restriction, or fix this.
-  WmWindowMus::Get(window)->SetBounds(*bounds);
-  *bounds = window->bounds();
-  return true;
+void WindowManager::OnWmSetBounds(aura::Window* window,
+                                  const gfx::Rect& bounds) {
+  WmWindowMus::Get(window)->SetBounds(bounds);
 }
 
 bool WindowManager::OnWmSetProperty(
     aura::Window* window,
     const std::string& name,
     std::unique_ptr<std::vector<uint8_t>>* new_data) {
   // TODO(sky): constrain this to set of keys we know about, and allowed values.
   if (name == ui::mojom::WindowManager::kWindowIgnoredByShelf_Property) {
     wm::WindowState* window_state =
         static_cast<WmWindow*>(WmWindowMus::Get(window))->GetWindowState();
@@ skipping 121 matching lines @@
 
 void WindowManager::OnWindowInitialized(aura::Window* window) {
   // This ensures WmWindowAura won't be called before WmWindowMus. This is
   // important as if WmWindowAura::Get() is called first, then WmWindowAura
   // would be created, not WmWindowMus.
   WmWindowMus::Get(window);
 }
 
 }  // namespace mus
 }  // namespace ash