[PDF] Fix font width generation when glyph 0 is used.

src/core/SkAdvancedTypefaceMetrics.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 "SkAdvancedTypefaceMetrics.h"
@@ skipping 151 matching lines @@
         firstIndex = static_cast<int>(subsetGlyphIDs[0]);
         lastIndex =
                 static_cast<int>(subsetGlyphIDs[subsetGlyphIDsLength - 1]) + 1;
     }
     curRange = appendRange(&result, firstIndex);
 
     for (int gId = firstIndex; gId <= lastIndex; gId++) {
         Data advance = kInvalidAdvance;
         if (gId < lastIndex) {
             // Get glyph id only when subset is NULL, or the id is in subset.
+            SkASSERT(!subsetGlyphIDs || (subsetIndex < subsetGlyphIDsLength &&
+                    static_cast<uint32_t>(gId) <= subsetGlyphIDs[subsetIndex]));
             if (!subsetGlyphIDs ||
                 (subsetIndex < subsetGlyphIDsLength &&
                  static_cast<uint32_t>(gId) == subsetGlyphIDs[subsetIndex])) {
                 SkAssertResult(getAdvance(fontHandle, gId, &advance));
                 ++subsetIndex;
             } else {
                 advance = kDontCareAdvance;
             }
         }
         if (advance == lastAdvance) {
@@ skipping 124 matching lines @@
 
 // additional declaration needed for testing with a face of an unknown type
 template SkAdvancedTypefaceMetrics::WidthRange* getAdvanceData(
         void* fontData,
         int num_glyphs,
         const uint32_t* subsetGlyphIDs,
         uint32_t subsetGlyphIDsLength,
         bool (*getAdvance)(void* fontData, int gId, int16_t* data));
 
 } // namespace skia_advanced_typeface_metrics_utils