NOT FOR SUBMIT - Windows Views HiDPI

content/renderer/render_widget.cc

 // 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.
 
 #include "content/renderer/render_widget.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 1171 matching lines @@
     // Compute a buffer for painting and cache it.
 
     bool fractional_scale = device_scale_factor_ -
         static_cast<int>(device_scale_factor_) != 0;
     if (fractional_scale) {
       // Damage might not be DIP aligned. Inflate damage to compensate.
       bounds.Inset(-1, -1);
       bounds.Intersect(gfx::Rect(size_));
     }
 
-    gfx::Rect pixel_bounds = gfx::ToEnclosingRect(
+    gfx::Rect pixel_bounds = gfx::ToSizedRect(
         gfx::ScaleRect(bounds, device_scale_factor_));

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

Will need to test in Win-Aura with non-integer scales to ensure that we don't
break the Aura-backing store.  Having said that, I believe that enclosing rect
is a bit dangerous to use as it can be 1 pixel larger than the ceiled size.

 
     scoped_ptr<skia::PlatformCanvas> canvas(
         RenderProcess::current()->GetDrawingCanvas(&current_paint_buf_,
                                                    pixel_bounds));
     if (!canvas) {
       NOTREACHED();
       return;
     }
 
     // We may get back a smaller canvas than we asked for.
@@ skipping 1111 matching lines @@
 
   if (!context->Initialize(
           attributes,
           false /* bind generates resources */,
           CAUSE_FOR_GPU_LAUNCH_WEBGRAPHICSCONTEXT3DCOMMANDBUFFERIMPL_INITIALIZE))
     return NULL;
   return context.release();
 }
 
 }  // namespace content