Fix Refcount in FontDataCache for objects from FontFallbackIterator

third_party/WebKit/Source/platform/fonts/GlyphPageTreeNode.cpp

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 230 matching lines @@
     bool haveGlyphs;
     if (!fontData->isSegmented()) {
         m_page = GlyphPage::createForSingleFontData(this, toSimpleFontData(fontData));
         haveGlyphs = fill(m_page.get(), 0, GlyphPage::size, buffer, bufferLength, toSimpleFontData(fontData));
     } else {
         m_page = GlyphPage::createForMixedFontData(this);
         haveGlyphs = false;
 
         const SegmentedFontData* segmentedFontData = toSegmentedFontData(fontData);
         for (int i = segmentedFontData->numFaces() - 1; i >= 0; i--) {
-            const FontDataForRangeSet& fontDataForRangeSet = segmentedFontData->faceAt(i);
-            RefPtr<UnicodeRangeSet> ranges = fontDataForRangeSet.ranges();
+            RefPtr<FontDataForRangeSet> fontDataForRangeSet = segmentedFontData->faceAt(i);
+            RefPtr<UnicodeRangeSet> ranges = fontDataForRangeSet->ranges();
             // If there are no ranges, that means this font should be used for
             // the full codepoint range, thus running the loop once over a
             // synthetic full UnicodeRange object. Otherwise we use the ranges
             // that come from the segmented font.  This needs a locally
             // initialized size_t variable (or a cast alternatively) as a
             // compile fix for Android since size_t differs between platforms.
             const size_t oneRoundForZeroRanges = 1u;
             for (size_t i = 0; i < max(ranges->size(), oneRoundForZeroRanges); ++i) {
                 UnicodeRange range(0, kMaxCodepoint);
                 if (ranges->size()) {
                     range = ranges->rangeAt(i);
                 }
                 // all this casting is to ensure all the parameters to min and max have the same type,
                 // to avoid ambiguous template parameter errors on Windows
                 int from = max(0, static_cast<int>(range.from()) - static_cast<int>(start));
                 int to = 1 + min(static_cast<int>(range.to()) - static_cast<int>(start), static_cast<int>(GlyphPage::size) - 1);
                 if (from >= static_cast<int>(GlyphPage::size) || to <= 0)
                     continue;
 
                 // If this is a custom font needs to be loaded, do not fill
                 // the page so that font fallback is used while loading.
-                RefPtr<CustomFontData> customData = fontDataForRangeSet.fontData()->customFontData();
+                RefPtr<CustomFontData> customData = fontDataForRangeSet->fontData()->customFontData();
                 if (customData && customData->isLoadingFallback()) {
                     for (int j = from; j < to; j++) {
                         m_page->setCustomFontToLoad(j, customData.get());
                         haveGlyphs = true;
                     }
                     continue;
                 }
 
-                haveGlyphs |= fill(m_page.get(), from, to - from, buffer + from * (start < 0x10000 ? 1 : 2), (to - from) * (start < 0x10000 ? 1 : 2), fontDataForRangeSet.fontData());
+                haveGlyphs |= fill(m_page.get(), from, to - from, buffer + from * (start < 0x10000 ? 1 : 2), (to - from) * (start < 0x10000 ? 1 : 2), fontDataForRangeSet->fontData());
             }
         }
     }
 
     if (!haveGlyphs)
         m_page = nullptr;
 }
 
 void GlyphPageTreeNode::initializeOverridePage(const FontData* fontData, unsigned pageNumber)
 {
@@ skipping 159 matching lines @@
     // entries may use different fonts depending on character. If the Font
     // ever finds it needs a glyph out of the system fallback page, it will
     // ask the system for the best font to use and fill that glyph in for us.
     if (GlyphPage* parentPage = m_parent->page())
         return parentPage->createCopiedSystemFallbackPage(this);
     return GlyphPage::createForMixedFontData(this);
 }
 
 } // namespace blink