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

ui/aura/mus/window_tree_client.cc

 // Copyright 2014 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 "ui/aura/mus/window_tree_client.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 1284 matching lines @@
   // TODO(sky): this should likely route to WindowTreeHost.
   window_manager_delegate_->OnWmDisplayModified(display);
 }
 
 // TODO(riajiang): Convert between pixel and DIP for window bounds properly.
 // (http://crbug.com/646942)
 void WindowTreeClient::WmSetBounds(uint32_t change_id,
                                    Id window_id,
                                    const gfx::Rect& transit_bounds) {
   WindowMus* window = GetWindowByServerId(window_id);
-  bool result = false;
   if (window) {
     DCHECK(window_manager_delegate_);
-    gfx::Rect bounds = transit_bounds;
     // TODO: this needs to trigger scheduling a bounds change on |window|.
-    result =
-        window_manager_delegate_->OnWmSetBounds(window->GetWindow(), &bounds);
-    if (result) {
-      // If the resulting bounds differ return false. Returning false ensures
-      // the client applies the bounds we set below.
-      result = bounds == transit_bounds;
-      window->SetBoundsFromServer(bounds);
-    }
+    window_manager_delegate_->OnWmSetBounds(window->GetWindow(),
+                                            transit_bounds);
+    window->SetBoundsFromServer(transit_bounds);

[mfomitchev, 2016/12/07 20:59:46]

Ditto - remove

[thanhph, 2016/12/08 00:20:29]

Done.

   }
   if (window_manager_internal_client_)

[mfomitchev, 2016/12/07 20:59:46]

A comment similar to what you have in the Mus WTC would be nice here.

[thanhph, 2016/12/08 00:20:29]

Done.

-    window_manager_internal_client_->WmResponse(change_id, result);
+    window_manager_internal_client_->WmResponse(change_id, false);
 }
 
 void WindowTreeClient::WmSetProperty(
     uint32_t change_id,
     Id window_id,
     const std::string& name,
     const base::Optional<std::vector<uint8_t>>& transit_data) {
   WindowMus* window = GetWindowByServerId(window_id);
   bool result = false;
   if (window) {
@@ skipping 290 matching lines @@
   return ScheduleInFlightChange(base::MakeUnique<InFlightCaptureChange>(
       this, capture_synchronizer_.get(), window));
 }
 
 uint32_t WindowTreeClient::CreateChangeIdForFocus(WindowMus* window) {
   return ScheduleInFlightChange(base::MakeUnique<InFlightFocusChange>(
       this, focus_synchronizer_.get(), window));
 }
 
 }  // namespace aura