Unreviewed, rolling out r215175.
authorryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Apr 2017 17:28:02 +0000 (17:28 +0000)
committerryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Apr 2017 17:28:02 +0000 (17:28 +0000)
This change caused a flaky crash in existing media tests.

Reverted changeset:

"Add fallback fonts to video captions stylesheet."
https://bugs.webkit.org/show_bug.cgi?id=170495
http://trac.webkit.org/changeset/215175

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@215185 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/CaptionUserPreferencesMediaAF.cpp

index 8d688da..a7c010f 100644 (file)
@@ -1,3 +1,15 @@
+2017-04-10  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r215175.
+
+        This change caused a flaky crash in existing media tests.
+
+        Reverted changeset:
+
+        "Add fallback fonts to video captions stylesheet."
+        https://bugs.webkit.org/show_bug.cgi?id=170495
+        http://trac.webkit.org/changeset/215175
+
 2017-04-10  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         Data interaction on an image enclosed by an anchor should vend the anchor's URL
index 7d629f3..2d946fb 100644 (file)
@@ -83,8 +83,6 @@ SOFT_LINK_AVF_FRAMEWORK(CoreText)
 SOFT_LINK_AVF_FRAMEWORK_IMPORT(CoreText, CTFontDescriptorCopyAttribute,  CFTypeRef, (CTFontDescriptorRef descriptor, CFStringRef attribute), (descriptor, attribute));
 SOFT_LINK_AVF_POINTER(CoreText, kCTFontNameAttribute, CFStringRef)
 #define kCTFontNameAttribute getkCTFontNameAttribute()
-SOFT_LINK_AVF_POINTER(CoreText, kCTFontCascadeListAttribute, CFStringRef)
-#define kCTFontCascadeListAttribute getkCTFontCascadeListAttribute()
 
 #define CTFontDescriptorCopyAttribute softLink_CTFontDescriptorCopyAttribute
 
@@ -426,24 +424,13 @@ String CaptionUserPreferencesMediaAF::captionsDefaultFontCSS() const
     RetainPtr<CFTypeRef> name = adoptCF(CTFontDescriptorCopyAttribute(font.get(), kCTFontNameAttribute));
     if (!name)
         return emptyString();
-
+    
     StringBuilder builder;
     
     builder.append(getPropertyNameString(CSSPropertyFontFamily));
     builder.appendLiteral(": \"");
     builder.append(static_cast<CFStringRef>(name.get()));
     builder.append('"');
-
-    auto cascadeList = adoptCF(static_cast<CFArrayRef>(CTFontDescriptorCopyAttribute(font.get(), kCTFontCascadeListAttribute)));
-
-    for (CFIndex i = 0; i < CFArrayGetCount(cascadeList.get()); i++) {
-        auto fontCascade = static_cast<CTFontDescriptorRef>(CFArrayGetValueAtIndex(cascadeList.get(), i));
-        auto fontCascadeName = adoptCF(CTFontDescriptorCopyAttribute(fontCascade, kCTFontNameAttribute));
-        builder.append(", \"");
-        builder.append(static_cast<CFStringRef>(fontCascadeName.get()));
-        builder.append('"');
-    }
-    
     if (behavior == kMACaptionAppearanceBehaviorUseValue)
         builder.appendLiteral(" !important");
     builder.append(';');