ui/gfx: Allow for font-specific rendering settings.

ui/gfx/platform_font_win.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 "ui/gfx/platform_font_win.h"
 
 #include <windows.h>
 #include <math.h>
 
 #include <algorithm>
 #include <string>
 
 #include "base/logging.h"
 #include "base/strings/string_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/win/scoped_gdi_object.h"
 #include "base/win/scoped_hdc.h"
 #include "base/win/scoped_select_object.h"
 #include "base/win/win_util.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/font.h"
+#include "ui/gfx/font_render_params.h"
 #include "ui/gfx/win/scoped_set_map_mode.h"
 
 namespace {
 
 // If the tmWeight field of a TEXTMETRIC structure has a value >= this, the
 // font is bold.
 const int kTextMetricWeightBold = 700;
 
 // Returns the minimum font size, using the minimum size callback, if set.
 int GetMinimumFontSize() {
@@ skipping 151 matching lines @@
                            &localized_font_name[0]);
   if (length <= 0)
     return GetFontName();
   return base::SysWideToUTF8(localized_font_name);
 }
 
 int PlatformFontWin::GetFontSize() const {
   return font_ref_->font_size();
 }
 
+const FontRenderParams& PlatformFontWin::GetFontRenderParams() const {
+  return GetDefaultFontRenderParams();

[msw, 2014/07/12 00:27:33]

nit: Can we have a static (lazy init) struct to return here?

[msw, 2014/07/12 00:27:33]

Should there be a TODO here for any per-font settings?

[Daniel Erat, 2014/07/12 01:55:33]

the windows implementation of GetDefaultFontRenderParams() returns a reference
to a lazily-initialized singleton, if i'm reading your code correctly :-P

[Daniel Erat, 2014/07/12 01:55:33]

i wasn't sure if windows supports this. font_render_params_win.cc looks like
it's grabbing global settings via SystemParametersInfo().

[msw, 2014/07/12 02:26:41]

Ah, right.

[msw, 2014/07/12 02:26:41]

Perhaps it's not supported, it's fine as-is without a TODO.

+}
+
 NativeFont PlatformFontWin::GetNativeFont() const {
   return font_ref_->hfont();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Font, private:
 
 void PlatformFontWin::InitWithCopyOfHFONT(HFONT hfont) {
   DCHECK(hfont);
   LOGFONT font_info;
@@ skipping 165 matching lines @@
   return new PlatformFontWin(native_font);
 }
 
 // static
 PlatformFont* PlatformFont::CreateFromNameAndSize(const std::string& font_name,
                                                   int font_size) {
   return new PlatformFontWin(font_name, font_size);
 }
 
 }  // namespace gfx