Apply hinting in SkiaTextRenderer::SetFontRenderParams().

ui/gfx/font_render_params_win.cc

 // Copyright 2014 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 "ui/gfx/font_render_params.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/singleton.h"
 #include "ui/gfx/win/singleton_hwnd.h"
 
 namespace gfx {
 
 namespace {
 
 // Caches font render params and updates them on system notifications.
 class CachedFontRenderParams : public gfx::SingletonHwnd::Observer {
  public:
   static CachedFontRenderParams* GetInstance() {
     return Singleton<CachedFontRenderParams>::get();
   }
 
   const FontRenderParams& GetParams() {
     if (params_)
       return *params_;
 
     params_.reset(new FontRenderParams());
     params_->antialiasing = false;
     params_->subpixel_positioning = false;
+    params_->autohinter = false;
+    params_->use_bitmaps = false;
+    params_->hinting = FontRenderParams::HINTING_MEDIUM;

[Daniel Erat, 2014/07/10 23:46:35]

we weren't setting these explicitly here, but i believe that these are the
default skia settings

[msw, 2014/07/11 00:29:09]

Acknowledged.

     params_->subpixel_rendering = FontRenderParams::SUBPIXEL_RENDERING_NONE;
 
     BOOL enabled = false;
     if (SystemParametersInfo(SPI_GETFONTSMOOTHING, 0, &enabled, 0) && enabled) {
       params_->antialiasing = true;
       params_->subpixel_positioning = true;
 
       UINT type = 0;
       if (SystemParametersInfo(SPI_GETFONTSMOOTHINGTYPE, 0, &type, 0) &&
           type == FE_FONTSMOOTHINGCLEARTYPE) {
@@ skipping 32 matching lines @@
 
 const FontRenderParams& GetDefaultFontRenderParams() {
   return CachedFontRenderParams::GetInstance()->GetParams();
 }
 
 const FontRenderParams& GetDefaultWebKitFontRenderParams() {
   return GetDefaultFontRenderParams();
 }
 
 }  // namespace gfx