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

services/ui/public/cpp/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 "services/ui/public/cpp/window_tree_client.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 24 matching lines @@
 
 namespace ui {
 
 Id MakeTransportId(ClientSpecificId client_id, ClientSpecificId local_id) {
   return (client_id << 16) | local_id;
 }
 
 // Helper function to get the device_scale_factor() of the display::Display
 // with |display_id|.
 float ScaleFactorForDisplay(int64_t display_id) {
-  // TODO(jonross): Remove this once aura-mus is complete. Currently the screen
-  // is not being set properly for mus-in-renderer. (http://crbug.com/659155)
-  if (!display::Screen::GetScreen())
-    return 1.f;
-
   // TODO(riajiang): Change to use display::GetDisplayWithDisplayId() after
   // https://codereview.chromium.org/2361283002/ is landed.
   std::vector<display::Display> displays =
       display::Screen::GetScreen()->GetAllDisplays();
   auto iter = std::find_if(displays.begin(), displays.end(),
                            [display_id](const display::Display& display) {
                              return display.id() == display_id;
                            });
   if (iter != displays.end())
     return iter->device_scale_factor();
@@ skipping 1254 matching lines @@
 
 void WindowTreeClient::WmSetBounds(uint32_t change_id,
                                        Id window_id,
                                        const gfx::Rect& transit_bounds) {
   Window* window = GetWindowByServerId(window_id);
   bool result = false;
   if (window) {
     DCHECK(window_manager_delegate_);
     gfx::Rect transit_bounds_in_dip = gfx::ConvertRectToDIP(
         ScaleFactorForDisplay(window->display_id()), transit_bounds);
-    gfx::Rect bounds = transit_bounds_in_dip;
-    result = window_manager_delegate_->OnWmSetBounds(window, &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_in_dip;
-      window->SetBounds(bounds);
-    }
+    window_manager_delegate_->OnWmSetBounds(window, &transit_bounds_in_dip);

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

Add window->SetBounds(transit_bounds_in_dip)?

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

Done.

+    result = true;
   }
   if (window_manager_internal_client_)
     window_manager_internal_client_->WmResponse(change_id, result);
 }
 
 void WindowTreeClient::WmSetProperty(
     uint32_t change_id,
     Id window_id,
     const std::string& name,
     const base::Optional<std::vector<uint8_t>>& transit_data) {
@@ skipping 136 matching lines @@
     // TODO(riajiang): Figure out if |offset| needs to be converted.
     // (http://crbugs.com/646932)
     window_manager_internal_client_->SetUnderlaySurfaceOffsetAndExtendedHitArea(
         server_id(window), offset.x(), offset.y(),
         gfx::ConvertInsetsToDIP(ScaleFactorForDisplay(window->display_id()),
                                 hit_area));
   }
 }
 
 }  // namespace ui