Chromium Code Reviews| Index: tests/FontConfigParser.cpp |
| diff --git a/tests/FontConfigParser.cpp b/tests/FontConfigParser.cpp |
| index dc1fac0547ba44b0eb09fac6dd7ee99b5aecf34b..373ba41cf98ac0c8c64f0a56cdedd1907076f1a6 100644 |
| --- a/tests/FontConfigParser.cpp |
| +++ b/tests/FontConfigParser.cpp |
| @@ -48,6 +48,17 @@ void ValidateLoadedFonts(SkTDArray<FontFamily*> fontFamilies, const char* firstE |
| REPORTER_ASSERT(reporter, isALPHA(c) || isDIGIT(c) || '-' == c); |
| } |
| } |
| + |
| + // All file names in the test configuration files start with a capital letter. |
|
djsollen
2015/02/13 16:17:49
comment that this isn't a requirement, but since i
bungeman-skia
2015/02/13 16:38:07
Done.
|
| + for (int i = 0; i < fontFamilies.count(); ++i) { |
| + FontFamily& family = *fontFamilies[i]; |
| + for (int j = 0; j < family.fFonts.count(); ++j) { |
| + FontFileInfo& file = family.fFonts[j]; |
| + REPORTER_ASSERT(reporter, !file.fFileName.isEmpty() && |
| + file.fFileName[0] >= 'A' && |
| + file.fFileName[0] <= 'Z'); |
| + } |
| + } |
| } |
| void DumpLoadedFonts(SkTDArray<FontFamily*> fontFamilies) { |