Simplify WindowManager::OnWmSetBounds

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/observer_list.h"
 #include "base/strings/utf_string_conversions.h"
 #include "mash/simple_wm/move_event_handler.h"
 #include "ui/aura/client/aura_constants.h"
@@ skipping 393 matching lines @@
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // SimpleWM, aura::WindowManagerDelegate implementation:
 
 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 false;
+  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;
 }
 
 void SimpleWM::OnWmSetModalType(aura::Window* window, ui::ModalType type) {}
@@ skipping 159 matching lines @@
 }
 
 void SimpleWM::OnWindowListViewItemActivated(aura::Window* window) {
   window->Show();
   aura::client::ActivationClient* activation_client =
       aura::client::GetActivationClient(window->GetRootWindow());
   activation_client->ActivateWindow(window);
 }
 
 }  // namespace simple_wm