Simplify WindowManager::OnWmSetBounds

services/ui/ws/window_tree.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/ws/window_tree.h"
 
 #include <stddef.h>
 
 #include <utility>
 
@@ skipping 2052 matching lines @@
       window->SetBounds(window_server_->GetCurrentMoveLoopRevertBounds(),
                         base::nullopt);
     }
 
     window_server_->EndMoveLoop();
   }
 
   window_server_->WindowManagerChangeCompleted(change_id, response);
 }
 
+void WindowTree::WmSetBoundsResponse(uint32_t change_id) {
+  WmResponse(change_id, false);

[sky, 2017/03/27 19:22:51]

Document why false is used.

[Fady Samuel, 2017/03/27 20:31:11]

Done.

+}
+
 void WindowTree::WmRequestClose(Id transport_window_id) {
   ServerWindow* window =
       GetWindowByClientId(ClientWindowId(transport_window_id));
   WindowTree* tree = window_server_->GetTreeWithRoot(window);
   if (tree && tree != this) {
     tree->client()->RequestClose(tree->ClientWindowIdForWindow(window).id);
   }
   // TODO(sky): think about what else case means.
 }
 
@@ skipping 189 matching lines @@
   client()->OnCompleteDrop(client_window_id.id, event_flags, cursor_offset,
                            effect_bitmask, callback);
 }
 
 void WindowTree::PerformOnDragDropDone() {
   client()->OnDragDropDone();
 }
 
 }  // namespace ws
 }  // namespace ui