Provide user friendly messages for OTS parsing of fonts

Source/platform/fonts/FontCustomPlatformData.cpp

Index: Source/platform/fonts/FontCustomPlatformData.cpp
diff --git a/Source/platform/fonts/FontCustomPlatformData.cpp b/Source/platform/fonts/FontCustomPlatformData.cpp
index 1cae8696dde0622b8f6c7993a978996dd3b131e5..fe7dd97331c08ef3e0daa559a8d6b21df8a7711b 100644
--- a/Source/platform/fonts/FontCustomPlatformData.cpp
+++ b/Source/platform/fonts/FontCustomPlatformData.cpp
@@ -85,14 +85,17 @@ FontPlatformData FontCustomPlatformData::fontPlatformData(float size, bool bold,
     return FontPlatformData(m_typeface.get(), "", size, bold && !m_typeface->isBold(), italic && !m_typeface->isItalic(), orientation);
 }
 
-PassOwnPtr<FontCustomPlatformData> FontCustomPlatformData::create(SharedBuffer* buffer)
+PassOwnPtr<FontCustomPlatformData> FontCustomPlatformData::create(SharedBuffer* buffer, String& otsParseMessage)
 {
     ASSERT_ARG(buffer, buffer);
 
     OpenTypeSanitizer sanitizer(buffer);
     RefPtr<SharedBuffer> transcodeBuffer = sanitizer.sanitize();
-    if (!transcodeBuffer)
+
+    if (!transcodeBuffer) {

[jungshik at Google, 2015/05/15 19:17:20]

I took a look at sanitize() to find that there are two  cases where
|transcodeBuffer == nullptr} but |m_parsingError| is uninitialized. So checking
|!transcodeBuffer| makes sense. 

[h.joshi, 2015/05/18 05:58:28]

Method "setParsingError" was not called at right place. Corrected that in new
patch.

+        otsParseMessage = sanitizer.getErrorString();
         return nullptr; // validation failed.
+    }
     buffer = transcodeBuffer.get();
 
     SkMemoryStream* stream = new SkMemoryStream(buffer->getAsSkData().get());