Remove implicit flooring Scale() method from Point and Size.

chrome/browser/thumbnails/thumbnail_tab_helper.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 "chrome/browser/thumbnails/thumbnail_tab_helper.h"
 
 #include "base/metrics/histogram.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/history/top_sites.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/thumbnails/render_widget_snapshot_taker.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "skia/ext/platform_canvas.h"
 #include "ui/gfx/color_utils.h"
+#include "ui/gfx/size_conversions.h"
 #include "ui/gfx/screen.h"
 #include "ui/gfx/scrollbar_size.h"
 #include "ui/gfx/skbitmap_operations.h"
 
 #if defined(OS_WIN)
 #include "base/win/windows_version.h"
 #endif
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(ThumbnailTabHelper)
 
@@ skipping 50 matching lines @@
 // ui::SCALE_FACTOR_200P because the high resolution thumbnail on high density
 // display alleviates the aliasing.
 // TODO(mazda): Copy the pixels with the smaller size in the case of
 // ui::SCALE_FACTOR_100P once the resampling method has been improved.
 gfx::Size GetCopySizeForThumbnail(content::RenderWidgetHostView* view) {
   gfx::Size copy_size(kThumbnailWidth, kThumbnailHeight);
   ui::ScaleFactor scale_factor =
       ui::GetScaleFactorForNativeView(view->GetNativeView());
   switch (scale_factor) {
     case ui::SCALE_FACTOR_100P:
-      copy_size =
-          copy_size.Scale(ui::GetScaleFactorScale(ui::SCALE_FACTOR_200P));
+      copy_size = gfx::ToFlooredSize(
+          copy_size.Scale(ui::GetScaleFactorScale(ui::SCALE_FACTOR_200P)));
       break;
     case ui::SCALE_FACTOR_200P:
       // Use the size as-is.
       break;
     default:
       DLOG(WARNING) << "Unsupported scale factor. Use the same copy size as "
                     << "ui::SCALE_FACTOR_100P";
-      copy_size =
-          copy_size.Scale(ui::GetScaleFactorScale(ui::SCALE_FACTOR_200P));
+      copy_size = gfx::ToFlooredSize(
+          copy_size.Scale(ui::GetScaleFactorScale(ui::SCALE_FACTOR_200P)));
       break;
   }
   return copy_size;
 }
 
 // Returns the size of the thumbnail stored in the database in pixel.
 gfx::Size GetThumbnailSizeInPixel() {
   gfx::Size thumbnail_size(kThumbnailWidth, kThumbnailHeight);
   // Determine the resolution of the thumbnail based on the primary monitor.
   // TODO(oshima): Use device's default scale factor.
   gfx::Display primary_display = gfx::Screen::GetPrimaryDisplay();
-  return thumbnail_size.Scale(primary_display.device_scale_factor());
+  return gfx::ToFlooredSize(
+      thumbnail_size.Scale(primary_display.device_scale_factor()));
 }
 
 // Returns the clipping rectangle that is used for creating a thumbnail with
 // the size of |desired_size| from the bitmap with the size of |source_size|.
 // The type of clipping that needs to be done is assigned to |clip_result|.
 gfx::Rect GetClippingRect(const gfx::Size& source_size,
                           const gfx::Size& desired_size,
                           ThumbnailTabHelper::ClipResult* clip_result) {
   DCHECK(clip_result);
 
@@ skipping 353 matching lines @@
 void ThumbnailTabHelper::DidStartLoading(
     content::RenderViewHost* render_view_host) {
   load_interrupted_ = false;
 }
 
 void ThumbnailTabHelper::StopNavigation() {
   // This function gets called when the page loading is interrupted by the
   // stop button.
   load_interrupted_ = true;
 }