Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1250)

Unified Diff: tests/FontConfigParser.cpp

Issue 887113002: Add factory for Android font manager. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Remove no longer wanted function. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/ports/SkFontMgr_android.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/FontConfigParser.cpp
diff --git a/tests/FontConfigParser.cpp b/tests/FontConfigParser.cpp
index ff72708be001ff6a4635bf2acd31cd12851735f1..744a41e5f2900c770ec4e6c1a620796beed53dbe 100644
--- a/tests/FontConfigParser.cpp
+++ b/tests/FontConfigParser.cpp
@@ -37,6 +37,9 @@ void DumpLoadedFonts(SkTDArray<FontFamily*> fontFamilies) {
case kCompact_FontVariant: SkDebugf(" compact\n"); break;
default: break;
}
+ if (fontFamilies[i]->fBasePath) {
+ SkDebugf(" basePath %s\n", fontFamilies[i]->fBasePath);
+ }
if (!fontFamilies[i]->fLanguage.getTag().isEmpty()) {
SkDebugf(" language %s\n", fontFamilies[i]->fLanguage.getTag().c_str());
}
@@ -56,7 +59,8 @@ DEF_TEST(FontConfigParserAndroid, reporter) {
bool resourcesMissing = false;
SkTDArray<FontFamily*> preV17FontFamilies;
- SkFontConfigParser::GetTestFontFamilies(preV17FontFamilies,
+ SkFontConfigParser::GetCustomFontFamilies(preV17FontFamilies,
+ SkString("/custom/font/path/"),
GetResourcePath("android_fonts/pre_v17/system_fonts.xml").c_str(),
GetResourcePath("android_fonts/pre_v17/fallback_fonts.xml").c_str());
@@ -72,7 +76,8 @@ DEF_TEST(FontConfigParserAndroid, reporter) {
SkTDArray<FontFamily*> v17FontFamilies;
- SkFontConfigParser::GetTestFontFamilies(v17FontFamilies,
+ SkFontConfigParser::GetCustomFontFamilies(v17FontFamilies,
+ SkString("/custom/font/path/"),
GetResourcePath("android_fonts/v17/system_fonts.xml").c_str(),
GetResourcePath("android_fonts/v17/fallback_fonts.xml").c_str());
@@ -88,7 +93,8 @@ DEF_TEST(FontConfigParserAndroid, reporter) {
SkTDArray<FontFamily*> v22FontFamilies;
- SkFontConfigParser::GetTestFontFamilies(v22FontFamilies,
+ SkFontConfigParser::GetCustomFontFamilies(v22FontFamilies,
+ SkString("/custom/font/path/"),
GetResourcePath("android_fonts/v22/fonts.xml").c_str(),
NULL);
« no previous file with comments | « src/ports/SkFontMgr_android.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698