Simplify WindowManager::OnWmSetBounds

services/ui/test_wm/test_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 <memory>
 #include <utility>
 
 #include "base/memory/ptr_util.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "mojo/public/cpp/bindings/binding.h"
@@ skipping 80 matching lines @@
     // Don't care.
   }
   aura::PropertyConverter* GetPropertyConverter() override {
     return &property_converter_;
   }
 
   // aura::WindowManagerDelegate:
   void SetWindowManagerClient(aura::WindowManagerClient* client) override {
     window_manager_client_ = client;
   }
-  bool OnWmSetBounds(aura::Window* window, gfx::Rect* bounds) override {
-    window->SetBounds(*bounds);
-    return true;
+  void OnWmSetBounds(aura::Window* window, const gfx::Rect& bounds) override {
+    window->SetBounds(bounds);
   }
   bool OnWmSetProperty(
       aura::Window* window,
       const std::string& name,
       std::unique_ptr<std::vector<uint8_t>>* new_data) override {
     return true;
   }
   void OnWmSetModalType(aura::Window* window, ui::ModalType type) override {}
   void OnWmSetCanFocus(aura::Window* window, bool can_focus) override {}
   aura::Window* OnWmCreateTopLevelWindow(
@@ skipping 83 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(TestWM);
 };
 
 }  // namespace test
 }  // namespace ui
 
 MojoResult ServiceMain(MojoHandle service_request_handle) {
   service_manager::ServiceRunner runner(new ui::test::TestWM);
   return runner.Run(service_request_handle);
 }