commit edb10e79f5cd1b48baa77f82705d4e711fb35ba4
author bungeman <bungeman@google.com> Mon Jul 28 08:19:45 2014 -0700
committer Commit bot <commit-bot@chromium.org> Mon Jul 28 08:19:45 2014 -0700
tree a079ece38ad38c8409eb437f7e9e7fd4bd5b68a1
parent 5a27f49a87718bd0e766e79d26de33c829145f13

Only lowercase names in Android font config parser.

BUG=chromium:397549
R=tomhudson@chromium.org, tomhudson@google.com

Author: bungeman@google.com

Review URL: https://codereview.chromium.org/420223003

src/ports/SkFontConfigParser_android.cpp

diff --git a/src/ports/SkFontConfigParser_android.cpp b/src/ports/SkFontConfigParser_android.cpp
index fc8e919..cc2ca8c 100644
--- a/src/ports/SkFontConfigParser_android.cpp
+++ b/src/ports/SkFontConfigParser_android.cpp
@@ -51,15 +51,16 @@
  * or file tag. The resulting strings are put into the fNames or FontFileInfo arrays.
  */
 static void textHandler(void *data, const char *s, int len) {
-    SkAutoAsciiToLC tolc(s);
     FamilyData *familyData = (FamilyData*) data;
     // Make sure we're in the right state to store this name information
     if (familyData->currentFamily &&
             (familyData->currentTag == NAMESET_TAG || familyData->currentTag == FILESET_TAG)) {
         switch (familyData->currentTag) {
-        case NAMESET_TAG:
+        case NAMESET_TAG: {
+            SkAutoAsciiToLC tolc(s, len);
             familyData->currentFamily->fNames.push_back().set(tolc.lc(), len);
             break;
+        }
         case FILESET_TAG:
             if (familyData->currentFontInfo) {
                 familyData->currentFontInfo->fFileName.set(s, len);