NOT FOR SUBMIT - Windows Views HiDPI

chrome/browser/thumbnails/simple_thumbnail_crop.cc

 // Copyright (c) 2013 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/simple_thumbnail_crop.h"
 
 #include "base/metrics/histogram.h"
 #include "content/public/browser/browser_thread.h"
 #include "skia/ext/platform_canvas.h"
 #include "ui/gfx/color_utils.h"
@@ skipping 110 matching lines @@
       copy_size = gfx::ToFlooredSize(gfx::ScaleSize(
           copy_size, 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 = gfx::ToFlooredSize(gfx::ScaleSize(
+#if defined(HIGHDPI)
+          copy_size, ui::GetScaleFactorScale(ui::SCALE_FACTOR_140P)));

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

ui::GetMaxScaleFactor() in place of the if-else?

[girard, 2013/06/03 22:31:55]

Good idea. Done.

+#else
           copy_size, ui::GetScaleFactorScale(ui::SCALE_FACTOR_200P)));
+#endif
       break;
   }
   return copy_size;
 }
 
 SimpleThumbnailCrop::~SimpleThumbnailCrop() {
 }
 
 // Returns the size of the thumbnail stored in the database in pixel.
 gfx::Size SimpleThumbnailCrop::GetThumbnailSizeInPixel() const {
@@ skipping 95 matching lines @@
       clipped_bitmap.height() == result.height())
     clipped_bitmap.copyTo(&result, SkBitmap::kARGB_8888_Config);
 #endif
 
   HISTOGRAM_TIMES(kThumbnailHistogramName,
                   base::TimeTicks::Now() - begin_compute_thumbnail);
   return result;
 }
 
 }