NOT FOR SUBMIT - Windows Views HiDPI

ui/gfx/rect_conversions.cc

Index: ui/gfx/rect_conversions.cc
diff --git a/ui/gfx/rect_conversions.cc b/ui/gfx/rect_conversions.cc
index 251c03711ddd9c670b9fc8c7148a59959e537e5a..2ad96e8da5e45a509a1570cf9d73e7118af7d097 100644
--- a/ui/gfx/rect_conversions.cc
+++ b/ui/gfx/rect_conversions.cc
@@ -11,6 +11,14 @@
 
 namespace gfx {
 
+Rect ToSizedRect(const RectF& rect) {

[kevers, 2013/04/19 20:31:42]

This logic is identical to ToFlooredRectDeprecated, which is typically not
appropriate for non-integer scale factors.  Non-integer-scaled assets use ceiled
size. If this method is to address issues with off-by-one errors in the
dimensions of the backing store, then consider using ToFlooredPoint combined
with ToCeiledSize.  Rect has a constructor Rect(Point,Size).  Consider using it
to achieve a more compact representation of the scaling operation.

+  int min_x = ToFlooredInt(rect.origin().x());
+  int min_y = ToFlooredInt(rect.origin().y());
+  int width = ToFlooredInt(rect.size().width());
+  int height = ToFlooredInt(rect.size().height());
+  return Rect(min_x, min_y, width, height);
+}
+
 Rect ToEnclosingRect(const RectF& rect) {
   int min_x = ToFlooredInt(rect.x());
   int min_y = ToFlooredInt(rect.y());