Fix root ServerWindow size.

services/ui/ws/display.h

 // 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.
 
 #ifndef SERVICES_UI_WS_DISPLAY_H_
 #define SERVICES_UI_WS_DISPLAY_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 119 matching lines @@
 
   // Called just before |tree| is destroyed.
   void OnWillDestroyTree(WindowTree* tree);
 
   void UpdateNativeCursor(mojom::Cursor cursor_id);
 
   // mojom::WindowTreeHost:
   void SetSize(const gfx::Size& size) override;
   void SetTitle(const mojo::String& title) override;
 
-  // Updates the root window, if necessary, for viewport metric changes.
-  void OnViewportMetricsChanged(const display::ViewportMetrics& new_metrics);
+  // Updates the size of display root ServerWindow and WM ServerWindow(s) for

[msw, 2016/11/09 20:57:18]

nit: "the display's root ServerWindow"? and "for the |new_size|", fix the other
mention of |size| to |new_size|.

[kylechar, 2016/11/09 21:36:56]

Changed method otherwise.

+  // new |size| if necessary. |size| is in physical pixels.
+  void UpdateRootWindowSize(const gfx::Size& new_size);

[msw, 2016/11/09 20:57:18]

Would it make sense to keep this as OnViewportMetricsChanged, and just have the
impl use its pixel_size?

[kylechar, 2016/11/09 21:36:56]

Hmm, we're not using the rest of the values now.. but we definitely need them in
the future. Changed back.

 
  private:
   friend class test::DisplayTestApi;
 
   using WindowManagerDisplayRootMap =
       std::map<UserId, WindowManagerDisplayRoot*>;
 
   // Inits the necessary state once the display is ready.
   void InitWindowManagerDisplayRootsIfNecessary();
 
@@ skipping 46 matching lines @@
 
   WindowManagerDisplayRootMap window_manager_display_root_map_;
 
   DISALLOW_COPY_AND_ASSIGN(Display);
 };
 
 }  // namespace ws
 }  // namespace ui
 
 #endif  // SERVICES_UI_WS_DISPLAY_H_