Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(815)

Unified Diff: content/renderer/render_widget.cc

Issue 11081007: Remove implicit flooring Scale() method from Point and Size. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_widget.cc
diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index a2edba1fbfb6dc03e10843bb7fb6549d7701565f..406b53181c3ddd9c32fd442ff1de60a44ec3aa7c 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -38,6 +38,7 @@
#include "ui/gfx/point.h"
#include "ui/gfx/rect_conversions.h"
#include "ui/gfx/size.h"
+#include "ui/gfx/size_conversions.h"
#include "ui/gfx/skia_util.h"
#include "ui/gl/gl_switches.h"
#include "ui/surface/transport_dib.h"
@@ -1494,8 +1495,10 @@ void RenderWidget::OnMsgPaintAtSize(const TransportDIB::Handle& dib_handle,
scoped_ptr<TransportDIB> paint_at_size_buffer(
TransportDIB::CreateWithHandle(dib_handle));
- gfx::Size page_size_in_pixel = page_size.Scale(device_scale_factor_);
- gfx::Size desired_size_in_pixel = desired_size.Scale(device_scale_factor_);
+ gfx::Size page_size_in_pixel = gfx::ToFlooredSize(
+ page_size.Scale(device_scale_factor_));
+ gfx::Size desired_size_in_pixel = gfx::ToFlooredSize(
+ desired_size.Scale(device_scale_factor_));
gfx::Size canvas_size = page_size_in_pixel;
float x_scale = static_cast<float>(desired_size_in_pixel.width()) /
static_cast<float>(canvas_size.width());
« no previous file with comments | « content/renderer/browser_plugin/browser_plugin_backing_store.cc ('k') | content/renderer/render_widget_fullscreen_pepper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698