Changeset 195567 in webkit for trunk/Source/WebCore/css/CSSFontFaceSource.cpp

Timestamp:

Jan 25, 2016, 4:35:17 PM (9 years ago)

Author:

[email protected]

Message:

Remove broken cache from CSSFontFaceSource
​https://bugs.webkit.org/show_bug.cgi?id=153440

Reviewed by Simon Fraser.

This cache has been broken since 2013 (r158085). Given we didn't notice a perf
hit when it broke, and the fact it's been broken for years, it clearly isn't
necessary.

​https://bugs.webkit.org/show_bug.cgi?id=153414 consists of a fairly invasive
change to CSSFontFaceSource; this patch includes a working version of this
cache, along with an easy way to enable/disable it (to measure possible perf
changes).

This patch is a short-term cleanup patch in the mean time until the above
invasive change gets landed.

No new tests because there is no behavior (or performance!) change.

• css/CSSFontFaceSource.cpp:

(WebCore::CSSFontFaceSource::font):
(WebCore::CSSFontFaceSource::~CSSFontFaceSource): Deleted.
(WebCore::CSSFontFaceSource::pruneTable): Deleted.
(WebCore::CSSFontFaceSource::fontLoaded): Deleted.

• css/CSSFontFaceSource.h:

File:

• trunk/Source/WebCore/css/CSSFontFaceSource.cpp (modified) (6 diffs)

trunk/Source/WebCore/css/CSSFontFaceSource.cpp

@@ -67 +67 @@
     if (m_font)
         m_font->removeClient(this);
-    pruneTable();
-}
-
-void CSSFontFaceSource::pruneTable()
-{
-    if (m_fontTable.isEmpty())
-        return;
-
-    m_fontTable.clear();
 }
 
@@ -87 +78 @@
 void CSSFontFaceSource::fontLoaded(CachedFont*)
 {
-    pruneTable();
     if (m_face)
         m_face->fontLoaded(this);
@@ -108 +98 @@
     }
 
-    unsigned hashKey = (fontDescription.computedPixelSize() + 1) << 5 | fontDescription.widthVariant() << 3
-                       | (fontDescription.orientation() == Vertical ? 4 : 0) | (syntheticBold ? 2 : 0) | (syntheticItalic ? 1 : 0);
-
-    RefPtr<Font> font = m_fontTable.add(hashKey, nullptr).iterator->value;
-    if (font)
-        return font.release();
-
     if (!m_font || m_font->isLoaded()) {
         if (m_font) {
@@ -120 +103 @@
                 return nullptr;
 
-            font = m_font->createFont(fontDescription, m_string, syntheticBold, syntheticItalic, fontFaceFeatures, fontFaceVariantSettings);
+            return m_font->createFont(fontDescription, m_string, syntheticBold, syntheticItalic, fontFaceFeatures, fontFaceVariantSettings);
         } else {
 #if ENABLE(SVG_FONTS)
@@ -138 +121 @@
                 if (!customPlatformData)
                     return nullptr;
-                font = Font::create(customPlatformData->fontPlatformData(fontDescription, syntheticBold, syntheticItalic, fontFaceFeatures, fontFaceVariantSettings), true, false);
+                return Font::create(customPlatformData->fontPlatformData(fontDescription, syntheticBold, syntheticItalic, fontFaceFeatures, fontFaceVariantSettings), true, false);
 #else
-                font = Font::create(std::make_unique<SVGFontData>(m_svgFontFaceElement.get()), fontDescription.computedPixelSize(), syntheticBold, syntheticItalic);
+                return Font::create(std::make_unique<SVGFontData>(m_svgFontFaceElement.get()), fontDescription.computedPixelSize(), syntheticBold, syntheticItalic);
 #endif
             }
 #endif
+            return nullptr;
         }
     } else {
@@ -150 +134 @@
         fontSelector->beginLoadingFontSoon(m_font.get());
 
-        Ref<Font> placeholderFont = FontCache::singleton().lastResortFallbackFont(fontDescription);
-        Ref<Font> placeholderFontCopyInLoadingState = Font::create(placeholderFont->platformData(), true, true);
-        return WTFMove(placeholderFontCopyInLoadingState);
+        return Font::create(FontCache::singleton().lastResortFallbackFont(fontDescription)->platformData(), true, true);
     }
-
-    return font.release();
 }