Check whether font creation is failed to avoid segment fault.

tests/FontHostStreamTest.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #include "SkTypes.h"
 
 #include "Test.h"
 #include "SkBitmap.h"
@@ skipping 54 matching lines @@
     return true;
 }
 
 static void test_fontHostStream(skiatest::Reporter* reporter) {
 
     {
         SkPaint paint;
         paint.setColor(SK_ColorGRAY);
         paint.setTextSize(SkIntToScalar(30));
 
-        paint.setTypeface(SkTypeface::CreateFromName("Georgia", SkTypeface::kNormal))->unref();
+        SkTypeface* font =

[bungeman-skia, 2013/05/03 14:11:41]

I think this just needs to be

SkSafeUnref(paint.setTypeface(SkTypeface::CreateFromName("Georgia",
SkTypeface::kNormal)))

The possibility of any font creation function returning NULL is somewhat of a
new experimental thing. We're currently working out the details.

+            SkTypeface::CreateFromName("Georgia", SkTypeface::kNormal);
+        if (NULL == font) {
+            printf("create font failed.\n");
+            return;
+        }
+        paint.setTypeface(font)->unref();
 
         SkIRect origRect = SkIRect::MakeWH(64, 64);
         SkBitmap origBitmap;
         create(&origBitmap, origRect, SkBitmap::kARGB_8888_Config);
         SkCanvas origCanvas(origBitmap);
 
         SkIRect streamRect = SkIRect::MakeWH(64, 64);
         SkBitmap streamBitmap;
         create(&streamBitmap, streamRect, SkBitmap::kARGB_8888_Config);
         SkCanvas streamCanvas(streamBitmap);
@@ skipping 19 matching lines @@
 
         REPORTER_ASSERT(reporter,
                         compare(origBitmap, origRect, streamBitmap, streamRect));
     }
     //Make sure the typeface is deleted and removed.
     SkGraphics::PurgeFontCache();
 }
 
 #include "TestClassDef.h"
 DEFINE_TESTCLASS("FontHost::CreateTypefaceFromStream", FontHostStreamTestClass, test_fontHostStream)