Use DirectWrite to populate list of avalible fonts

content/common/font_list_win.cc

 // Copyright (c) 2011 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 "content/common/font_list.h"
 
 #include <windows.h>
+#include <dwrite.h>
 
 #include <set>
 
+#include "base/debug/alias.h"
+#include "base/logging.h"
 #include "base/strings/string16.h"
 #include "base/values.h"
+#include "content/common/sandbox_win.h"
+#include "content/public/renderer/render_font_warmup_win.h"
 
 namespace content {
 
 static int CALLBACK EnumFontFamExProc(ENUMLOGFONTEXW* logical_font,
                                       NEWTEXTMETRICEXW* physical_font,
                                       DWORD font_type,
                                       LPARAM lparam) {
   std::set<base::string16>* font_names =
       reinterpret_cast<std::set<base::string16>*>(lparam);
   if (font_names) {
     const LOGFONTW& lf = logical_font->elfLogFont;
     if (lf.lfFaceName[0] && lf.lfFaceName[0] != '@') {
       base::string16 face_name(lf.lfFaceName);
       font_names->insert(face_name);
     }
   }
   return 1;
 }
 
-scoped_ptr<base::ListValue> GetFontList_SlowBlocking() {
+static inline void GetFontListInternal_GDI(
+    scoped_ptr<base::ListValue>& font_list) {
   std::set<base::string16> font_names;
 
   LOGFONTW logfont;
   memset(&logfont, 0, sizeof(logfont));
   logfont.lfCharSet = DEFAULT_CHARSET;
 
   HDC hdc = ::GetDC(NULL);
   ::EnumFontFamiliesExW(hdc, &logfont, (FONTENUMPROCW)&EnumFontFamExProc,
                         (LPARAM)&font_names, 0);
   ::ReleaseDC(NULL, hdc);
 
-  scoped_ptr<base::ListValue> font_list(new base::ListValue);
   std::set<base::string16>::iterator iter;
   for (iter = font_names.begin(); iter != font_names.end(); ++iter) {
     base::ListValue* font_item = new base::ListValue();
     font_item->Append(new base::StringValue(*iter));
     font_item->Append(new base::StringValue(*iter));
     font_list->Append(font_item);
   }
+}
+
+static inline void AddLocalizedFontFamily(

[scottmg, 2014/06/26 20:13:38]

nit; no 'inline's necessary

+    scoped_ptr<base::ListValue>& font_list,
+    IDWriteFontFamily* font_family,
+    wchar_t* user_locale) {
+  IDWriteLocalizedStrings* family_names = NULL;
+  if (SUCCEEDED(font_family->GetFamilyNames(&family_names))) {
+    UINT32 index = 0;
+    BOOL exists = false;
+
+    family_names->FindLocaleName(user_locale, &index, &exists);
+    if (!exists) {
+      family_names->FindLocaleName(L"en-us", &index, &exists);
+    }
+    if (!exists) {
+      index = 0;
+    }
+
+    UINT32 len = 0;
+    if (SUCCEEDED(family_names->GetStringLength(index, &len))) {
+      wchar_t* name = new wchar_t[len + 1];
+      if (name) {
+        if (SUCCEEDED(family_names->GetString(index, name, len + 1))) {
+          base::ListValue* font_item = new base::ListValue();
+          font_item->Append(new base::StringValue(name));
+          font_item->Append(new base::StringValue(name));
+          font_list->Append(font_item);
+        }
+        delete[] name;
+      }
+    }
+  }
+  if (family_names) {
+    family_names->Release();
+  }
+}
+
+static inline void GetFontListInternal_DirectWrite(
+    scoped_ptr<base::ListValue>& font_list) {
+  wchar_t user_locale[LOCALE_NAME_MAX_LENGTH];
+  GetUserDefaultLocaleName(user_locale, LOCALE_NAME_MAX_LENGTH);
+
+  IDWriteFactory* factory = NULL;
+  CreateDirectWriteFactory(&factory);
+
+  IDWriteFontCollection* font_collection;
+  BOOL checkForUpdates = false;
+  if (SUCCEEDED(factory->GetSystemFontCollection(&font_collection,
+      checkForUpdates))) {
+    UINT32 family_count = font_collection->GetFontFamilyCount();
+    for (UINT32 i = 0; i < family_count; i++) {
+      IDWriteFontFamily* font_family = NULL;
+      if (SUCCEEDED(font_collection->GetFontFamily(i, &font_family))) {
+        AddLocalizedFontFamily(font_list, font_family, user_locale);

[scottmg, 2014/06/26 20:13:38]

I'm not sure how this changes the actual names returned -- will we need to
update various .grd translations of font names along with this?

[eae, 2014/06/26 20:58:24]

For fonts with multiple names (e.x. SimSun which is either "宋体" or "SimSun")
this will pick show the name that matches to current locale, falling back on
English or the first name listed.

Given that the names are interchangeable from a programmatic standpoint I don't
think we need to change the translations but I must confess that I don't know
much about the chrome translations code.

[scottmg, 2014/06/26 21:05:17]

OK, I just noticed e.g. in
https://src.chromium.org/viewvc/chrome?revision=279449&view=revision the .grd
includes the name? Maybe cpu can comment here.

+      }
+      if (font_family) {
+        font_family->Release();
+      }
+    }
+  }
+  if (font_collection) {
+    font_collection->Release();
+  }
+  if (factory) {
+    factory->Release();
+  }
+}
+
+scoped_ptr<base::ListValue> GetFontList_SlowBlocking() {

[scottmg, 2014/06/26 20:13:38]

I'm a little unclear on where this happens. If it's in the browser (I assume?),
then we shouldn't be using the content/public/renderer code.

[eae, 2014/06/26 20:58:24]

Yeah, the way I get the factory from the sandbox code is a complete hack right
now. Not quite sure what the right way to get the DW factory is though.

+  scoped_ptr<base::ListValue> font_list(new base::ListValue);
+  if (ShouldUseDirectWrite()) {
+    GetFontListInternal_DirectWrite(font_list);
+  }
+  if (font_list->GetSize() < 1) {
+    GetFontListInternal_GDI(font_list);
+  }
   return font_list.Pass();
 }
 
 }  // namespace content