Force all font backends to override onGetFontDescriptor, so we can

src/ports/SkFontHost_android.cpp

 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkFontHost.h"
 #include "SkFontHost_FreeType_common.h"
 #include "SkFontDescriptor.h"
@@ skipping 298 matching lines @@
             remove_from_names(family);
             detach_and_delete_family(family);
         }
     }
 
     bool isSysFont() const { return fIsSysFont; }
 
     virtual const char* getUniqueString() const = 0;
     virtual const char* getFilePath() const = 0;
 
+protected:
+    virtual void onGetFontDescriptor(SkFontDescriptor*, bool*) const SK_OVERRIDE;
+
 private:
     bool    fIsSysFont;
 
     typedef SkTypeface_FreeType INHERITED;
 };
 
 ///////////////////////////////////////////////////////////////////////////////
 
 class StreamTypeface : public FamilyTypeface {
 public:
@@ skipping 401 matching lines @@
         init_system_fonts();
     } else if (strncmp(locale.language, prevLocale.language, 2) ||
             strncmp(locale.region, prevLocale.region, 2)) {
         prevLocale = locale;
         reload_fallback_fonts();
     }
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
-void SkFontHost::Serialize(const SkTypeface* face, SkWStream* stream) {
-
-    SkFontDescriptor descriptor;
+void FamilyTypeface::onGetFontDescriptor(SkFontDescriptor* desc,
+                                         bool* isLocalStream) const {
     {
         SkAutoMutexAcquire ac(gFamilyHeadAndNameListMutex);
-        descriptor.setFamilyName(find_family_name(face));
-        descriptor.setStyle(face->style());
-        descriptor.setFontFileName(((FamilyTypeface*)face)->getUniqueString());
+        desc->setFamilyName(find_family_name(this));
+        desc->setFontFileName(this->getUniqueString());
     }
-
-    descriptor.serialize(stream);
-
-    const bool isCustomFont = !((FamilyTypeface*)face)->isSysFont();
-    if (isCustomFont) {
-        // store the entire font in the fontData
-        SkStream* fontStream = face->openStream(NULL);
-        const uint32_t length = fontStream->getLength();
-
-        stream->writePackedUInt(length);
-        stream->writeStream(fontStream, length);
-
-        fontStream->unref();
-    } else {
-        stream->writePackedUInt(0);
-    }
+    *isLocalStream = !this->isSysFont();
 }
 
+#if 0   // do we need this different name lookup for Deserialize?

[bungeman-skia, 2013/03/22 21:04:05]

Should we ask djsollen about this?

[reed1, 2013/03/25 12:26:27]

Done.

 SkTypeface* SkFontHost::Deserialize(SkStream* stream) {
-    {
-        SkAutoMutexAcquire  ac(gFamilyHeadAndNameListMutex);
-        load_system_fonts();
-    }
-
-    SkFontDescriptor descriptor(stream);
-    const char* familyName = descriptor.getFamilyName();
-    const char* fontFileName = descriptor.getFontFileName();
-    const SkTypeface::Style style = descriptor.getStyle();
-
-    const uint32_t customFontDataLength = stream->readPackedUInt();
-    if (customFontDataLength > 0) {
-
-        // generate a new stream to store the custom typeface
-        SkMemoryStream* fontStream = new SkMemoryStream(customFontDataLength - 1);
-        stream->read((void*)fontStream->getMemoryBase(), customFontDataLength - 1);
-
-        SkTypeface* face = CreateTypefaceFromStream(fontStream);
-
-        fontStream->unref();
-        return face;
-    }
-
+    ...
     if (NULL != fontFileName && 0 != *fontFileName) {
         const FontInitRec* rec = gSystemFonts;
         for (size_t i = 0; i < gNumSystemFonts; i++) {
             if (strcmp(rec[i].fFileName, fontFileName) == 0) {
                 // backup until we hit the fNames
                 for (int j = i; j >= 0; --j) {
                     if (rec[j].fNames != NULL) {
                         return SkFontHost::CreateTypeface(NULL,
                                     rec[j].fNames[0], style);
                     }
                 }
             }
         }
     }
-
-    return SkFontHost::CreateTypeface(NULL, familyName, style);
+    ...
 }
+#endif
 
 ///////////////////////////////////////////////////////////////////////////////
 
 SkTypeface* SkFontHost::CreateTypeface(const SkTypeface* familyFace,
                                        const char familyName[],
                                        SkTypeface::Style style) {
     SkAutoMutexAcquire  ac(gFamilyHeadAndNameListMutex);
 
     load_system_fonts();
 
@@ skipping 273 matching lines @@
                              const char* fontsdir) {
     gTestMainConfigFile = mainconf;
     gTestFallbackConfigFile = fallbackconf;
     gTestFontFilePrefix = fontsdir;
     SkASSERT(gTestMainConfigFile);
     SkASSERT(gTestFallbackConfigFile);
     SkASSERT(gTestFontFilePrefix);
     SkDEBUGF(("Use Test Config File Main %s, Fallback %s, Font Dir %s",
               gTestMainConfigFile, gTestFallbackConfigFile, gTestFontFilePrefix));
 }