Fix FontMgrTest to allow for unnamed fonts.

tests/FontMgrTest.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkCommandLineFlags.h"
 #include "SkFontMgr.h"
 #include "SkTypeface.h"
@@ skipping 29 matching lines @@
     }
 }
 
 static void test_fontiter(skiatest::Reporter* reporter, bool verbose) {
     SkAutoTUnref<SkFontMgr> fm(SkFontMgr::RefDefault());
     int count = fm->countFamilies();
 
     for (int i = 0; i < count; ++i) {
         SkString fname;
         fm->getFamilyName(i, &fname);
-        REPORTER_ASSERT(reporter, fname.size() > 0);
-
+        if (fname.size() == 0) {

[bungeman-skia, 2014/03/05 18:01:53]

Ack! A tab!

Do we even need this? Does anything fail if fname.size() is 0? We should be able
to that case.

[hal.canary, 2014/03/05 18:24:32]

Now that I think about it, the other ASSERTs didn't trigger, so we can skip this
test.

+            continue;
+        }
         SkAutoTUnref<SkFontStyleSet> fnset(fm->matchFamily(fname.c_str()));
         SkAutoTUnref<SkFontStyleSet> set(fm->createStyleSet(i));
         REPORTER_ASSERT(reporter, fnset->count() == set->count());
 
         if (verbose) {
             SkDebugf("[%2d] %s\n", i, fname.c_str());
         }
 
         for (int j = 0; j < set->count(); ++j) {
             SkString sname;
@@ skipping 11 matching lines @@
         }
     }
 }
 
 DEFINE_bool(verboseFontMgr, false, "run verbose fontmgr tests.");
 
 DEF_TEST(FontMgr, reporter) {
     test_fontiter(reporter, FLAGS_verboseFontMgr);
     test_alias_names(reporter);
 }