Retrieve font metrics from skia if DirectWrite is enabled for font metrics calculations in the chro…

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/macros.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_comptr.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 "third_party/skia/include/core/SkTypeface.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;
@@ skipping 42 matching lines @@
 
 // static
 PlatformFontWin::HFontRef* PlatformFontWin::base_font_ref_;
 
 // static
 PlatformFontWin::AdjustFontCallback
     PlatformFontWin::adjust_font_callback = NULL;
 PlatformFontWin::GetMinimumFontSizeCallback
     PlatformFontWin::get_minimum_font_size_callback = NULL;
 
-IDWriteFactory* PlatformFontWin::direct_write_factory_ = NULL;
+bool PlatformFontWin::use_skia_for_font_metrics_ = false;
 
 ////////////////////////////////////////////////////////////////////////////////
 // PlatformFontWin, public
 
 PlatformFontWin::PlatformFontWin() : font_ref_(GetBaseFontRef()) {
 }
 
 PlatformFontWin::PlatformFontWin(NativeFont native_font) {
   InitWithCopyOfHFONT(native_font);
 }
@@ skipping 147 matching lines @@
     base_font_ref_ = PlatformFontWin::CreateHFontRef(font);
     // base_font_ref_ is global, up the ref count so it's never deleted.
     base_font_ref_->AddRef();
   }
   return base_font_ref_;
 }
 
 PlatformFontWin::HFontRef* PlatformFontWin::CreateHFontRef(HFONT font) {
   TEXTMETRIC font_metrics;
 
-  if (direct_write_factory_) {
-    base::win::ScopedGDIObject<HFONT> gdi_font(font);
-    font = ConvertGDIFontToDirectWriteFont(gdi_font);
-  }
+  if (use_skia_for_font_metrics_)
+    return CreateHFontRefFromSkia(font);
 
   {
     base::win::ScopedGetDC screen_dc(NULL);
     gfx::ScopedSetMapMode mode(screen_dc, MM_TEXT);
     GetTextMetricsForFont(screen_dc, font, &font_metrics);
   }
 
   return CreateHFontRef(font, font_metrics);
 }
 
@@ skipping 44 matching lines @@
     if (font_metrics.tmHeight > best_font_metrics.tmHeight)
       break;
     best_font.Set(font.release());
     best_font_metrics = font_metrics;
   } while (true);
 
   return Font(new PlatformFontWin(CreateHFontRef(best_font.release())));
 }
 
 // static
-HFONT PlatformFontWin::ConvertGDIFontToDirectWriteFont(HFONT gdi_font) {
-  // This function uses  the DirectWrite Gdi interop interfaces to convert the
-  // gdi font passed in to a HFONT which is compatible with DirectWrite font
-  // metrics which could be different from GDI font metrics. This ensures that
-  // widgets like labels which use font metrics to calculate bounds have the
-  // same calculations as skia which uses DirectWrite.
-  DCHECK(direct_write_factory_);
-  base::win::ScopedComPtr<IDWriteGdiInterop> gdi_interop;
-  HRESULT hr = direct_write_factory_->GetGdiInterop(gdi_interop.Receive());
-  if (FAILED(hr)) {
-    CHECK(false);
-    return NULL;
+PlatformFontWin::HFontRef* PlatformFontWin::CreateHFontRefFromSkia(
+    HFONT gdi_font) {
+  LOGFONT font_info = {0};
+  GetObject(gdi_font, sizeof(LOGFONT), &font_info);
+
+  int skia_style = SkTypeface::kNormal;
+  if (font_info.lfWeight >= FW_SEMIBOLD &&
+      font_info.lfWeight <= FW_ULTRABOLD) {
+    skia_style |= SkTypeface::kBold;
   }
+  if (font_info.lfItalic)
+    skia_style |= SkTypeface::kItalic;
 
-  base::win::ScopedGetDC screen_dc(NULL);
-  gfx::ScopedSetMapMode mode(screen_dc, MM_TEXT);
+  skia::RefPtr<SkTypeface> skia_face = skia::AdoptRef(
+      SkTypeface::CreateFromName(
+          base::SysWideToUTF8(font_info.lfFaceName).c_str(),
+                              static_cast<SkTypeface::Style>(skia_style)));
+  SkPaint paint;
+  paint.setAntiAlias(true);

[scottmg, 2014/11/01 00:39:15]

is there anywhere these can be tied back to where the rendering happens to make
sure they match?

[ananta, 2014/11/01 00:56:13]

Used the SystemParametersInfo function on the same lines as the
CachedFontRenderParams class in font_render_params_win.cc.

+  paint.setTypeface(skia_face.get());
+  SkPaint::FontMetrics skia_metrics;
+  paint.getFontMetrics(&skia_metrics);
 
-  base::win::ScopedSelectObject scoped_font(screen_dc, gdi_font);
+  const int height =

[scottmg, 2014/11/01 00:39:15]

are all the max(1, ...) necessary? I'm not familiar with cap_height, any
reference for these calculations you could comment here?

[ananta, 2014/11/01 00:56:13]

This is similar to the CreateHFontRef function which gets these values
from GDI. Added a comment to that effect. 

+      std::max<int>(1, std::ceil(fabs(skia_metrics.fAscent)) +
+                    std::ceil(skia_metrics.fDescent));
+  const int baseline = std::max<int>(1, std::ceil(fabs(skia_metrics.fAscent)));
+  const int cap_height = std::max<int>(1,
+      std::ceil(fabs(skia_metrics.fAscent)) - skia_metrics.fLeading);
+  const int ave_char_width = std::max<int>(1, skia_metrics.fAvgCharWidth);
+  const int font_size = std::max<int>(1, height - skia_metrics.fLeading);
 
-  base::win::ScopedComPtr<IDWriteFontFace> font_face_gdi;
-  hr = gdi_interop->CreateFontFaceFromHdc(screen_dc, font_face_gdi.Receive());
-  if (FAILED(hr)) {
-    CHECK(false);
-    return NULL;
-  }
-
-  LOGFONT dwrite_to_gdi_log_font = {0};
-  hr = gdi_interop->ConvertFontFaceToLOGFONT(font_face_gdi,
-                                             &dwrite_to_gdi_log_font);
-  if (FAILED(hr)) {
-    CHECK(false);
-    return NULL;
-  }
-  return CreateFontIndirect(&dwrite_to_gdi_log_font);
+  int style = 0;
+  if (skia_style & SkTypeface::kItalic)
+    style |= Font::ITALIC;
+  if (font_info.lfUnderline)
+    style |= Font::UNDERLINE;
+  if (font_info.lfWeight >= kTextMetricWeightBold)
+    style |= Font::BOLD;
+  return new HFontRef(gdi_font, font_size, height, baseline, cap_height,
+                      ave_char_width, style);
 }
 
 PlatformFontWin::PlatformFontWin(HFontRef* hfont_ref) : font_ref_(hfont_ref) {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // PlatformFontWin::HFontRef:
 
 PlatformFontWin::HFontRef::HFontRef(HFONT hfont,
                                     int font_size,
@@ skipping 57 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