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

mash/simple_wm/simple_wm.cc

 // Copyright 2016 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 "mash/simple_wm/simple_wm.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/display/screen_base.h"
 #include "ui/gfx/canvas.h"
@@ skipping 120 matching lines @@
 
 aura::PropertyConverter* SimpleWM::GetPropertyConverter() {
   return &property_converter_;
 }
 
 void SimpleWM::SetWindowManagerClient(
     aura::WindowManagerClient* client) {
   window_manager_client_ = client;
 }
 
-bool SimpleWM::OnWmSetBounds(aura::Window* window, gfx::Rect* bounds) {
+void SimpleWM::OnWmSetBounds(aura::Window* window, const gfx::Rect& bounds) {
   FrameView* frame_view = GetFrameViewForClientWindow(window);
-  frame_view->GetWidget()->SetBounds(*bounds);
-  return true;
+  frame_view->GetWidget()->SetBounds(bounds);
 }
 
 bool SimpleWM::OnWmSetProperty(
     aura::Window* window,
     const std::string& name,
     std::unique_ptr<std::vector<uint8_t>>* new_data) {
   return true;
 }
 
 aura::Window* SimpleWM::OnWmCreateTopLevelWindow(
@@ skipping 81 matching lines @@
     const std::vector<gfx::Rect>& additional_client_areas) {}
 
 SimpleWM::FrameView* SimpleWM::GetFrameViewForClientWindow(
     aura::Window* client_window) {
   auto it = client_window_to_frame_view_.find(client_window);
   return it != client_window_to_frame_view_.end() ? it->second : nullptr;
 }
 
 }  // namespace simple_wm