Title: [206719] trunk

Diff

Modified: trunk/LayoutTests/ChangeLog (206718 => 206719)


--- trunk/LayoutTests/ChangeLog	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/ChangeLog	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,3 +1,18 @@
+2016-10-02  Chris Dumez  <[email protected]>
+
+        Unreviewed, rolling out r206692 and r206718.
+
+        Seems to have caused >200 failures on Sierra
+
+        Reverted changesets:
+
+        "Implement rendering of font-variation-settings"
+        https://bugs.webkit.org/show_bug.cgi?id=162782
+        http://trac.webkit.org/changeset/206692
+
+        "Tried to fix the build after r206692."
+        http://trac.webkit.org/changeset/206718
+
 2016-10-01  Simon Fraser  <[email protected]>
 
         Support transitions/animations of background-position with right/bottom-relative values

Deleted: trunk/LayoutTests/fast/text/variations/duplicate-expected.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/duplicate-expected.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/duplicate-expected.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia'; font-variation-settings: 'wght' 3;">Test passes if this text has a weight of 3.</div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/duplicate.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/duplicate.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/duplicate.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia'; font-variation-settings: 'wght' 2, 'wght' 3;">Test passes if this text has a weight of 3.</div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/exist-expected-mismatch.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/exist-expected-mismatch.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/exist-expected-mismatch.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia';">Test passes if this text is bold.</div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/exist.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/exist.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/exist.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: '-apple-system'; font-variation-settings: 'wght' 1.8;">Test passes if this text is bold.</div>
-</body>
-</html>

Modified: trunk/LayoutTests/fast/text/variations/getComputedStyle.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/getComputedStyle.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/getComputedStyle.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -29,4 +29,4 @@
 </script>
 <script src=""
 </body>
-</html>
\ No newline at end of file
+</html>

Deleted: trunk/LayoutTests/fast/text/variations/inheritance-expected.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/inheritance-expected.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/inheritance-expected.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia'; font-variation-settings: 'wght' 2;">Test passes if this text is bold.</div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/inheritance.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/inheritance.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/inheritance.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia'; font-variation-settings: 'wght' 2;"><div>Test passes if this text is bold.</div></div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/order-expected.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/order-expected.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/order-expected.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia'; font-variation-settings: 'wdth' 1.2, 'wght' 2;">Test passes if this text has a weight of 3.</div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/order.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/order.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/order.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia'; font-variation-settings: 'wght' 2, 'wdth' 1.2;">Test passes if this text has a weight of 3.</div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/outofbounds-expected.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/outofbounds-expected.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/outofbounds-expected.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia';">Test passes if this text is not bold.</div>
-</body>
-</html>
\ No newline at end of file

Deleted: trunk/LayoutTests/fast/text/variations/outofbounds.html (206718 => 206719)


--- trunk/LayoutTests/fast/text/variations/outofbounds.html	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/LayoutTests/fast/text/variations/outofbounds.html	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-<script>
-if (window.internals)
-   window.internals.settings.setVariationFontsEnabled(true);
-</script>
-</head>
-<body>
-<div style="font-family: 'Skia'; font-variation-settings: 'wght' 27;">Test passes if this text is not bold.</div>
-</body>
-</html>
\ No newline at end of file

Modified: trunk/Source/WebCore/ChangeLog (206718 => 206719)


--- trunk/Source/WebCore/ChangeLog	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/Source/WebCore/ChangeLog	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1,3 +1,18 @@
+2016-10-02  Chris Dumez  <[email protected]>
+
+        Unreviewed, rolling out r206692 and r206718.
+
+        Seems to have caused >200 failures on Sierra
+
+        Reverted changesets:
+
+        "Implement rendering of font-variation-settings"
+        https://bugs.webkit.org/show_bug.cgi?id=162782
+        http://trac.webkit.org/changeset/206692
+
+        "Tried to fix the build after r206692."
+        http://trac.webkit.org/changeset/206718
+
 2016-10-02  Dan Bernstein  <[email protected]>
 
         Tried to fix the build after r206692.

Modified: trunk/Source/WebCore/platform/graphics/FontCache.h (206718 => 206719)


--- trunk/Source/WebCore/platform/graphics/FontCache.h	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/Source/WebCore/platform/graphics/FontCache.h	2016-10-02 18:19:05 UTC (rev 206719)
@@ -259,7 +259,7 @@
     bool needsSyntheticOblique;
 };
 
-RetainPtr<CTFontRef> preparePlatformFont(CTFontRef, TextRenderingMode, const FontFeatureSettings* fontFaceFeatures, const FontVariantSettings* fontFaceVariantSettings, const FontFeatureSettings& features, const FontVariantSettings&, const FontVariationSettings&);
+RetainPtr<CTFontRef> preparePlatformFont(CTFontRef, TextRenderingMode, const FontFeatureSettings* fontFaceFeatures, const FontVariantSettings* fontFaceVariantSettings, const FontFeatureSettings& features, const FontVariantSettings&);
 FontWeight fontWeightFromCoreText(CGFloat weight);
 uint16_t toCoreTextFontWeight(FontWeight);
 bool isFontWeightBold(FontWeight);

Modified: trunk/Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp (206718 => 206719)


--- trunk/Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp	2016-10-02 18:19:05 UTC (rev 206719)
@@ -143,7 +143,6 @@
 }
 
 typedef HashMap<FontTag, int, FourCharacterTagHash, FourCharacterTagHashTraits> FeaturesMap;
-typedef HashMap<FontTag, float, FourCharacterTagHash, FourCharacterTagHashTraits> VariationsMap;
 
 static FeaturesMap computeFeatureSettingsFromVariants(const FontVariantSettings& variantSettings)
 {
@@ -361,9 +360,9 @@
     return result;
 }
 
-RetainPtr<CTFontRef> preparePlatformFont(CTFontRef originalFont, TextRenderingMode textRenderingMode, const FontFeatureSettings* fontFaceFeatures, const FontVariantSettings* fontFaceVariantSettings, const FontFeatureSettings& features, const FontVariantSettings& variantSettings, const FontVariationSettings& variations)
+RetainPtr<CTFontRef> preparePlatformFont(CTFontRef originalFont, TextRenderingMode textRenderingMode, const FontFeatureSettings* fontFaceFeatures, const FontVariantSettings* fontFaceVariantSettings, const FontFeatureSettings& features, const FontVariantSettings& variantSettings)
 {
-    if (!originalFont || (!features.size() && variations.isEmpty() && (textRenderingMode == AutoTextRendering) && variantSettings.isAllNormal()
+    if (!originalFont || (!features.size() && (textRenderingMode == AutoTextRendering) && variantSettings.isAllNormal()
         && (!fontFaceFeatures || !fontFaceFeatures->size()) && (!fontFaceVariantSettings || fontFaceVariantSettings->isAllNormal())))
         return originalFont;
 
@@ -399,12 +398,8 @@
     for (auto& newFeature : features)
         featuresToBeApplied.set(newFeature.tag(), newFeature.value());
 
-    VariationsMap variationsToBeApplied;
-    for (auto& newVariation : variations)
-        variationsToBeApplied.set(newVariation.tag(), newVariation.value());
-
     RetainPtr<CFMutableDictionaryRef> attributes = adoptCF(CFDictionaryCreateMutable(kCFAllocatorDefault, 2, &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks));
-    if (!featuresToBeApplied.isEmpty()) {
+    if (featuresToBeApplied.size()) {
         RetainPtr<CFMutableArrayRef> featureArray = adoptCF(CFArrayCreateMutable(kCFAllocatorDefault, features.size(), &kCFTypeArrayCallBacks));
         for (auto& p : featuresToBeApplied) {
             auto feature = FontFeature(p.key, p.value);
@@ -413,18 +408,6 @@
         }
         CFDictionaryAddValue(attributes.get(), kCTFontFeatureSettingsAttribute, featureArray.get());
     }
-
-    if (!variationsToBeApplied.isEmpty()) {
-        auto variationDictionary = adoptCF(CFDictionaryCreateMutable(kCFAllocatorDefault, 0, &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks));
-        for (auto& p : variationsToBeApplied) {
-            long long bitwiseTag = p.key[0] << 24 | p.key[1] << 16 | p.key[2] << 8 | p.key[3];
-            auto tagNumber = adoptCF(CFNumberCreate(kCFAllocatorDefault, kCFNumberLongLongType, &bitwiseTag));
-            auto valueNumber = adoptCF(CFNumberCreate(kCFAllocatorDefault, kCFNumberFloatType, &p.value));
-            CFDictionarySetValue(variationDictionary.get(), tagNumber.get(), valueNumber.get());
-        }
-        CFDictionaryAddValue(attributes.get(), kCTFontVariationAttribute, variationDictionary.get());
-    }
-
     if (textRenderingMode == OptimizeLegibility) {
         CGFloat size = CTFontGetSize(originalFont);
         RetainPtr<CFNumberRef> sizeNumber = adoptCF(CFNumberCreate(kCFAllocatorDefault, kCFNumberCGFloatType, &size));
@@ -693,7 +676,7 @@
 }
 #endif
 
-static RetainPtr<CTFontRef> fontWithFamily(const AtomicString& family, CTFontSymbolicTraits desiredTraits, FontWeight weight, const FontFeatureSettings& featureSettings, const FontVariantSettings& variantSettings, const FontVariationSettings& variationSettings, const FontFeatureSettings* fontFaceFeatures, const FontVariantSettings* fontFaceVariantSettings, const TextRenderingMode& textRenderingMode, float size)
+static RetainPtr<CTFontRef> fontWithFamily(const AtomicString& family, CTFontSymbolicTraits desiredTraits, FontWeight weight, const FontFeatureSettings& featureSettings, const FontVariantSettings& variantSettings, const FontFeatureSettings* fontFaceFeatures, const FontVariantSettings* fontFaceVariantSettings, const TextRenderingMode& textRenderingMode, float size)
 {
     if (family.isEmpty())
         return nullptr;
@@ -706,7 +689,7 @@
         foundFont = platformFontWithFamily(family, desiredTraits, weight, textRenderingMode, size);
 #endif
     }
-    return preparePlatformFont(foundFont.get(), textRenderingMode, fontFaceFeatures, fontFaceVariantSettings, featureSettings, variantSettings, variationSettings);
+    return preparePlatformFont(foundFont.get(), textRenderingMode, fontFaceFeatures, fontFaceVariantSettings, featureSettings, variantSettings);
 }
 
 #if PLATFORM(MAC)
@@ -746,7 +729,7 @@
     CTFontSymbolicTraits traits = computeTraits(fontDescription);
     float size = fontDescription.computedPixelSize();
 
-    RetainPtr<CTFontRef> font = fontWithFamily(family, traits, fontDescription.weight(), fontDescription.featureSettings(), fontDescription.variantSettings(), fontDescription.variationSettings(), fontFaceFeatures, fontFaceVariantSettings, fontDescription.textRenderingMode(), size);
+    RetainPtr<CTFontRef> font = fontWithFamily(family, traits, fontDescription.weight(), fontDescription.featureSettings(), fontDescription.variantSettings(), fontFaceFeatures, fontFaceVariantSettings, fontDescription.textRenderingMode(), size);
 
 #if PLATFORM(MAC)
     if (!font) {
@@ -757,7 +740,7 @@
         // Ignore the result because we want to use our own algorithm to actually find the font.
         autoActivateFont(family.string(), size);
 
-        font = fontWithFamily(family, traits, fontDescription.weight(), fontDescription.featureSettings(), fontDescription.variantSettings(), fontDescription.variationSettings(), fontFaceFeatures, fontFaceVariantSettings, fontDescription.textRenderingMode(), size);
+        font = fontWithFamily(family, traits, fontDescription.weight(), fontDescription.featureSettings(), fontDescription.variantSettings(), fontFaceFeatures, fontFaceVariantSettings, fontDescription.textRenderingMode(), size);
     }
 #endif
 
@@ -801,7 +784,7 @@
 
     const FontPlatformData& platformData = originalFontData->platformData();
     RetainPtr<CTFontRef> result = platformLookupFallbackFont(platformData.font(), description.weight(), description.locale(), characters, length);
-    result = preparePlatformFont(result.get(), description.textRenderingMode(), nullptr, nullptr, description.featureSettings(), description.variantSettings(), description.variationSettings());
+    result = preparePlatformFont(result.get(), description.textRenderingMode(), nullptr, nullptr, description.featureSettings(), description.variantSettings());
     if (!result)
         return lastResortFallbackFont(description);
 

Modified: trunk/Source/WebCore/platform/graphics/cocoa/FontPlatformDataCocoa.mm (206718 => 206719)


--- trunk/Source/WebCore/platform/graphics/cocoa/FontPlatformDataCocoa.mm	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/Source/WebCore/platform/graphics/cocoa/FontPlatformDataCocoa.mm	2016-10-02 18:19:05 UTC (rev 206719)
@@ -99,11 +99,11 @@
     return TextSpacingProportional;
 }
 
-static CFDictionaryRef cascadeToLastResortAttributesDictionary()
+static CTFontDescriptorRef cascadeToLastResortFontDescriptor()
 {
-    static CFDictionaryRef attributes = nullptr;
-    if (attributes)
-        return attributes;
+    static CTFontDescriptorRef descriptor;
+    if (descriptor)
+        return descriptor;
 
     RetainPtr<CTFontDescriptorRef> lastResort = adoptCF(CTFontDescriptorCreateWithNameAndSize(CFSTR("LastResort"), 0));
 
@@ -112,37 +112,11 @@
 
     const void* keys[] = { kCTFontCascadeListAttribute };
     const void* values[] = { array.get() };
-    attributes = CFDictionaryCreate(kCFAllocatorDefault, keys, values, WTF_ARRAY_LENGTH(keys), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks);
+    RetainPtr<CFDictionaryRef> attributes = adoptCF(CFDictionaryCreate(kCFAllocatorDefault, keys, values, WTF_ARRAY_LENGTH(keys), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks));
 
-    return attributes;
-}
+    descriptor = CTFontDescriptorCreateWithAttributes(attributes.get());
 
-static RetainPtr<CTFontDescriptorRef> cascadeToLastResortAndVariationsFontDescriptor(CTFontRef originalFont)
-{
-// FIXME: Remove this when <rdar://problem/28449441> is fixed.
-#define WORKAROUND_CORETEXT_VARIATIONS_WITH_FALLBACK_LIST_BUG ((PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED < 101300) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED < 110000))
-
-    CFDictionaryRef attributes = cascadeToLastResortAttributesDictionary();
-#if WORKAROUND_CORETEXT_VARIATIONS_WITH_FALLBACK_LIST_BUG
-    auto variations = adoptCF(static_cast<CFDictionaryRef>(CTFontCopyAttribute(originalFont, kCTFontVariationAttribute)));
-    if (!variations)
-#endif
-    {
-        UNUSED_PARAM(originalFont);
-
-        static CTFontDescriptorRef descriptor = nullptr;
-        if (descriptor)
-            return descriptor;
-
-        descriptor = CTFontDescriptorCreateWithAttributes(attributes);
-        return descriptor;
-    }
-#if WORKAROUND_CORETEXT_VARIATIONS_WITH_FALLBACK_LIST_BUG
-    auto mutableAttributes = adoptCF(CFDictionaryCreateMutableCopy(kCFAllocatorDefault, 2, attributes));
-    CFDictionaryAddValue(mutableAttributes.get(), kCTFontVariationAttribute, variations.get());
-    return adoptCF(CTFontDescriptorCreateWithAttributes(mutableAttributes.get()));
-#endif
-#undef WORKAROUND_CORETEXT_VARIATIONS_WITH_FALLBACK_LIST_BUG
+    return descriptor;
 }
 
 CTFontRef FontPlatformData::ctFont() const
@@ -154,7 +128,7 @@
 #if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED < 101200) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED < 100000)
     ASSERT(m_cgFont);
 #endif
-    m_ctFont = adoptCF(CTFontCreateCopyWithAttributes(m_font.get(), m_size, 0, cascadeToLastResortAndVariationsFontDescriptor(m_font.get()).get()));
+    m_ctFont = adoptCF(CTFontCreateCopyWithAttributes(m_font.get(), m_size, 0, cascadeToLastResortFontDescriptor()));
 
     if (m_widthVariant != RegularWidth) {
         int featureTypeValue = kTextSpacingType;

Modified: trunk/Source/WebCore/platform/graphics/mac/FontCustomPlatformData.cpp (206718 => 206719)


--- trunk/Source/WebCore/platform/graphics/mac/FontCustomPlatformData.cpp	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/Source/WebCore/platform/graphics/mac/FontCustomPlatformData.cpp	2016-10-02 18:19:05 UTC (rev 206719)
@@ -40,7 +40,7 @@
     FontOrientation orientation = fontDescription.orientation();
     FontWidthVariant widthVariant = fontDescription.widthVariant();
     RetainPtr<CTFontRef> font = adoptCF(CTFontCreateWithFontDescriptor(m_fontDescriptor.get(), size, nullptr));
-    font = preparePlatformFont(font.get(), fontDescription.textRenderingMode(), &fontFaceFeatures, &fontFaceVariantSettings, fontDescription.featureSettings(), fontDescription.variantSettings(), fontDescription.variationSettings());
+    font = preparePlatformFont(font.get(), fontDescription.textRenderingMode(), &fontFaceFeatures, &fontFaceVariantSettings, fontDescription.featureSettings(), fontDescription.variantSettings());
     ASSERT(font);
     return FontPlatformData(font.get(), size, bold, italic, orientation, widthVariant, fontDescription.textRenderingMode());
 }

Modified: trunk/Source/WebCore/rendering/RenderThemeIOS.mm (206718 => 206719)


--- trunk/Source/WebCore/rendering/RenderThemeIOS.mm	2016-10-02 14:49:13 UTC (rev 206718)
+++ trunk/Source/WebCore/rendering/RenderThemeIOS.mm	2016-10-02 18:19:05 UTC (rev 206719)
@@ -1278,7 +1278,7 @@
 
     ASSERT(fontDescriptor);
     RetainPtr<CTFontRef> font = adoptCF(CTFontCreateWithFontDescriptor(fontDescriptor.get(), 0, nullptr));
-    font = preparePlatformFont(font.get(), fontDescription.textRenderingMode(), nullptr, nullptr, fontDescription.featureSettings(), fontDescription.variantSettings(), fontDescription.variationSettings());
+    font = preparePlatformFont(font.get(), fontDescription.textRenderingMode(), nullptr, nullptr, fontDescription.featureSettings(), fontDescription.variantSettings());
     fontDescription.setIsAbsoluteSize(true);
     fontDescription.setOneFamily(textStyle);
     fontDescription.setSpecifiedSize(CTFontGetSize(font.get()));
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to