Chromium Code Reviews| Index: ppapi/shared_impl/private/ppb_browser_font_trusted_shared.cc |
| diff --git a/ppapi/shared_impl/private/ppb_browser_font_trusted_shared.cc b/ppapi/shared_impl/private/ppb_browser_font_trusted_shared.cc |
| index ab7c45acdc7012b58c9c5ec8cbefe2ef65f51224..529ba9eaa963478fba043c0d24933f9c1637bd7e 100644 |
| --- a/ppapi/shared_impl/private/ppb_browser_font_trusted_shared.cc |
| +++ b/ppapi/shared_impl/private/ppb_browser_font_trusted_shared.cc |
| @@ -4,6 +4,8 @@ |
| #include "ppapi/shared_impl/private/ppb_browser_font_trusted_shared.h" |
| +#include <unicode/uscript.h> |
| + |
| #include "base/string_util.h" |
| #include "base/utf_string_conversions.h" |
| #include "ppapi/c/dev/ppb_font_dev.h" |
| @@ -37,6 +39,16 @@ namespace ppapi { |
| namespace { |
| +const char *kCommonScript = uscript_getShortName(USCRIPT_COMMON); |
|
tony
2012/05/09 17:11:15
Nit: char*, not char *.
falken
2012/05/11 11:04:14
Done.
|
| + |
| +string16 GetFontFromMap(const WebPreferences::ScriptFontFamilyMap& map, |
| + const std::string& script) { |
| + WebPreferences::ScriptFontFamilyMap::const_iterator it = map.find(script); |
| + if (it != map.end()) |
| + return it->second; |
| + return string16(); |
| +} |
| + |
| bool PPTextRunToWebTextRun(const PP_BrowserFont_Trusted_TextRun& text, |
| WebTextRun* run) { |
| StringVar* text_string = StringVar::FromPPVar(text.text); |
| @@ -88,17 +100,21 @@ WebFontDescription PPFontDescToWebFontDesc( |
| // Resolve the generic family. |
| switch (font.family) { |
| case PP_BROWSERFONT_TRUSTED_FAMILY_SERIF: |
| - resolved_family = prefs.serif_font_family; |
| + resolved_family = GetFontFromMap(prefs.serif_font_family_map, |
| + kCommonScript); |
| break; |
| case PP_BROWSERFONT_TRUSTED_FAMILY_SANSSERIF: |
| - resolved_family = prefs.sans_serif_font_family; |
| + resolved_family = GetFontFromMap(prefs.sans_serif_font_family_map, |
| + kCommonScript); |
| break; |
| case PP_BROWSERFONT_TRUSTED_FAMILY_MONOSPACE: |
| - resolved_family = prefs.fixed_font_family; |
| + resolved_family = GetFontFromMap(prefs.fixed_font_family_map, |
| + kCommonScript); |
| break; |
| case PP_BROWSERFONT_TRUSTED_FAMILY_DEFAULT: |
| default: |
| - resolved_family = prefs.standard_font_family; |
| + resolved_family = GetFontFromMap(prefs.standard_font_family_map, |
| + kCommonScript); |
| break; |
| } |
| } else { |
| @@ -115,7 +131,8 @@ WebFontDescription PPFontDescToWebFontDesc( |
| // level to detect if the requested font is fixed width, so we only apply |
| // the alternate font size to the default fixed font family. |
| if (StringToLowerASCII(resolved_family) == |
| - StringToLowerASCII(prefs.fixed_font_family)) |
| + StringToLowerASCII(GetFontFromMap(prefs.fixed_font_family_map, |
| + kCommonScript))) |
| result.size = static_cast<float>(prefs.default_fixed_font_size); |
| else |
| result.size = static_cast<float>(prefs.default_font_size); |
| @@ -322,4 +339,3 @@ void PPB_BrowserFont_Trusted_Shared::DrawTextToCanvas( |
| } |
| } // namespace ppapi |
| - |