NOT FOR SUBMIT - Windows Views HiDPI

content/renderer/render_widget.cc

Index: content/renderer/render_widget.cc
diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index 39b7b3f98a9e60d56e5cebd3b12c99c6d5287d27..480371afc324f29c9537757987a9dcc7f29f1af0 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -50,6 +50,7 @@
 #include "third_party/skia/include/core/SkShader.h"
 #include "ui/base/ui_base_switches.h"
 #include "ui/gfx/point.h"
+#include "ui/gfx/point_conversions.h"
 #include "ui/gfx/rect_conversions.h"
 #include "ui/gfx/size_conversions.h"
 #include "ui/gfx/skia_util.h"
@@ -1235,9 +1236,11 @@ void RenderWidget::DoDeferredUpdate() {
       bounds.Inset(-1, -1);
       bounds.Intersect(gfx::Rect(size_));
     }
-
-    gfx::Rect pixel_bounds = gfx::ToEnclosingRect(
-        gfx::ScaleRect(bounds, device_scale_factor_));
+    // TODO: Migrate this sizing logic into backing store win/aura.
+    gfx::Rect pixel_bounds = gfx::Rect(

[kevers, 2013/06/03 17:28:50]

Looks like the necessary backing store changes are missing from this CL. The
bounds should not be updated independently of the backing stores. Suggest a
separate CL which adjusts for Windows and Aura so that the sizing is kept in
sync.

+        gfx::ToFlooredPoint(
+        gfx::ScalePoint(bounds.origin(),device_scale_factor_)),
+        gfx::ToCeiledSize(gfx::ScaleSize(bounds.size(),device_scale_factor_)));
 
     scoped_ptr<skia::PlatformCanvas> canvas(
         RenderProcess::current()->GetDrawingCanvas(&current_paint_buf_,