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

Unified Diff: tests/FontConfigParser.cpp

Issue 890073002: Update dump code in FontConfigParser test. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 11 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 | « no previous file | 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 4273b5eeab264173367a43fd78f749d05c69bbf2..ff72708be001ff6a4635bf2acd31cd12851735f1 100644
--- a/tests/FontConfigParser.cpp
+++ b/tests/FontConfigParser.cpp
@@ -33,24 +33,19 @@ void DumpLoadedFonts(SkTDArray<FontFamily*> fontFamilies) {
for (int i = 0; i < fontFamilies.count(); ++i) {
SkDebugf("Family %d:\n", i);
switch(fontFamilies[i]->fVariant) {
- case SkPaintOptionsAndroid::kElegant_Variant: SkDebugf(" elegant"); break;
- case SkPaintOptionsAndroid::kCompact_Variant: SkDebugf(" compact"); break;
+ case kElegant_FontVariant: SkDebugf(" elegant\n"); break;
+ case kCompact_FontVariant: SkDebugf(" compact\n"); break;
default: break;
}
if (!fontFamilies[i]->fLanguage.getTag().isEmpty()) {
- SkDebugf(" language: %s", fontFamilies[i]->fLanguage.getTag().c_str());
+ SkDebugf(" language %s\n", fontFamilies[i]->fLanguage.getTag().c_str());
}
for (int j = 0; j < fontFamilies[i]->fNames.count(); ++j) {
SkDebugf(" name %s\n", fontFamilies[i]->fNames[j].c_str());
}
for (int j = 0; j < fontFamilies[i]->fFonts.count(); ++j) {
const FontFileInfo& ffi = fontFamilies[i]->fFonts[j];
- SkDebugf(" file (%d %s %d) %s\n",
- ffi.fWeight,
- ffi.fPaintOptions.getLanguage().getTag().isEmpty() ? "" :
- ffi.fPaintOptions.getLanguage().getTag().c_str(),
- ffi.fPaintOptions.getFontVariant(),
- ffi.fFileName.c_str());
+ SkDebugf(" file (%d) %s#%d\n", ffi.fWeight, ffi.fFileName.c_str(), ffi.fIndex);
}
}
#endif // SK_DEBUG_FONTS
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698