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

services/ui/public/cpp/tests/window_server_test_base.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 "services/ui/public/cpp/tests/window_server_test_base.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
@@ skipping 100 matching lines @@
   DeleteWindowTreeClient(root->window_tree());
 }
 
 void WindowServerTestBase::OnPointerEventObserved(const ui::PointerEvent& event,
                                                   Window* target) {}
 
 void WindowServerTestBase::SetWindowManagerClient(WindowManagerClient* client) {
   window_manager_client_ = client;
 }
 
-bool WindowServerTestBase::OnWmSetBounds(Window* window, gfx::Rect* bounds) {
-  return window_manager_delegate_
-             ? window_manager_delegate_->OnWmSetBounds(window, bounds)
-             : true;
-}
+void WindowServerTestBase::OnWmSetBounds(Window* window, gfx::Rect* bounds) {}

[Fady Samuel, 2016/12/02 23:18:25]

This is changing behavior.

[thanhph, 2016/12/05 17:54:00]

Acknowledged.

 
 bool WindowServerTestBase::OnWmSetProperty(
     Window* window,
     const std::string& name,
     std::unique_ptr<std::vector<uint8_t>>* new_data) {
   return window_manager_delegate_
              ? window_manager_delegate_->OnWmSetProperty(window, name, new_data)
              : true;
 }
 
@@ skipping 56 matching lines @@
 }
 
 void WindowServerTestBase::Create(
     const service_manager::Identity& remote_identity,
     mojom::WindowTreeClientRequest request) {
   window_tree_clients_.insert(
       base::MakeUnique<WindowTreeClient>(this, nullptr, std::move(request)));
 }
 
 }  // namespace ui