Allow the default generic font family settings to find the first available font

chrome/browser/ui/webui/options/font_settings_utils.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_OPTIONS_FONT_SETTINGS_UTILS_H_
 #define CHROME_BROWSER_UI_WEBUI_OPTIONS_FONT_SETTINGS_UTILS_H_
 
+#include <string>
+
 #include "base/macros.h"
 
 class PrefService;
 
 namespace options {
 
 // Chrome advanced options utility methods.
 class FontSettingsUtilities {
  public:
   static void ValidateSavedFonts(PrefService* prefs);
 
+  // Returns the first available font if it starts with ",".

[msw, 2016/10/27 00:04:18]

This doesn't actually return a font, can you clarify the return value? Is it a
localized font family name? Also, clarify what this returns if the string does
not start with a comma. It would also be good to give the actual param name
instead of 'it', ie. "if |font_name_or_list| starts with".

[kojii, 2016/10/27 04:50:56]

Clarified. This function returns the specified name, the function below
|MaybeGetLocalizedFontName| gets localized name.

+  // Unlike gfx::FontList, this function picks one font, and character-level
+  // fallback is handled in CSS.
+  static std::string ResolveFontList(const std::string&);

[msw, 2016/10/27 00:04:18]

nit: add param names here and below.

[kojii, 2016/10/27 04:50:56]

Done.

+
+  // Returns the localized name of a font so that settings can find it within
+  // the list of system fonts. On Windows, the list of system fonts has names
+  // only for the system locale, but the pref value may be in the English name.
+  static std::string MaybeGetLocalizedFontName(const std::string&);
+
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(FontSettingsUtilities);
 };
 
 }  // namespace options
 
 #endif  // CHROME_BROWSER_UI_WEBUI_OPTIONS_FONT_SETTINGS_UTILS_H_