Refactor Windows DPI Point, Rect, and Size for Multiple Monitor DPI Awareness

ui/gfx/display.h

 // Copyright (c) 2012 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 UI_GFX_DISPLAY_H_
 #define UI_GFX_DISPLAY_H_
 
 #include <stdint.h>
 
 #include "base/compiler_specific.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/gfx_export.h"
 
 namespace gfx {
 
 // This class typically, but does not always, correspond to a physical display
 // connected to the system. A fake Display may exist on a headless system, or a
 // Display may correspond to a remote, virtual display.
 //
 // Note: The screen and display currently uses pixel coordinate
 // system. For platforms that support DIP (density independent pixel),
 // |bounds()| and |work_area| will return values in DIP coordinate
 // system, not in backing pixels.
+//
+// Display is designed to be copied, so don't inherit from this class!

[oshima, 2016/01/21 17:47:44]

can you make the class final instead? I believe it's allowed now.

[robliao, 2016/01/21 18:19:20]

Will do! That can be a separate CL too.

 class GFX_EXPORT Display {
  public:
   // Screen Rotation in clock-wise degrees.
   // This enum corresponds to DisplayRotationDefaultProto::Rotation in
   // chrome/browser/chromeos/policy/proto/chrome_device_policy.proto.
   enum Rotation {
     ROTATE_0 = 0,
     ROTATE_90,
     ROTATE_180,
     ROTATE_270,
@@ skipping 116 matching lines @@
   Rect bounds_;
   Rect work_area_;
   float device_scale_factor_;
   Rotation rotation_;
   TouchSupport touch_support_;
 };
 
 }  // namespace gfx
 
 #endif  // UI_GFX_DISPLAY_H_