Title: [193484] trunk/Source/WebCore
Revision
193484
Author
mmaxfi...@apple.com
Date
2015-12-04 15:52:28 -0800 (Fri, 04 Dec 2015)

Log Message

[Cocoa] Some fonts cause CTFontCopyAvailableTables() to return nullptr
https://bugs.webkit.org/show_bug.cgi?id=151884
<rdar://problem/23752253>

Reviewed by Brian Burg.

No new tests.

* platform/graphics/opentype/OpenTypeCG.cpp:
(WebCore::OpenType::fontHasMathTable):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (193483 => 193484)


--- trunk/Source/WebCore/ChangeLog	2015-12-04 23:22:29 UTC (rev 193483)
+++ trunk/Source/WebCore/ChangeLog	2015-12-04 23:52:28 UTC (rev 193484)
@@ -1,3 +1,16 @@
+2015-12-04  Myles C. Maxfield  <mmaxfi...@apple.com>
+
+        [Cocoa] Some fonts cause CTFontCopyAvailableTables() to return nullptr
+        https://bugs.webkit.org/show_bug.cgi?id=151884
+        <rdar://problem/23752253>
+
+        Reviewed by Brian Burg.
+
+        No new tests.
+
+        * platform/graphics/opentype/OpenTypeCG.cpp:
+        (WebCore::OpenType::fontHasMathTable):
+
 2015-12-04  Jer Noble  <jer.no...@apple.com>
 
         Unreviewed build-fix; Add new parameter to overloaded createSession() method in MediaPlayerPrivateAVFoundationCF.

Modified: trunk/Source/WebCore/platform/graphics/opentype/OpenTypeCG.cpp (193483 => 193484)


--- trunk/Source/WebCore/platform/graphics/opentype/OpenTypeCG.cpp	2015-12-04 23:22:29 UTC (rev 193483)
+++ trunk/Source/WebCore/platform/graphics/opentype/OpenTypeCG.cpp	2015-12-04 23:52:28 UTC (rev 193484)
@@ -39,6 +39,8 @@
 bool fontHasMathTable(CTFontRef ctFont)
 {
     RetainPtr<CFArrayRef> tableTags = adoptCF(CTFontCopyAvailableTables(ctFont, kCTFontTableOptionNoOptions));
+    if (!tableTags)
+        return false;
     CFIndex numTables = CFArrayGetCount(tableTags.get());
     for (CFIndex index = 0; index < numTables; ++index) {
         CTFontTableTag tag = (CTFontTableTag)(uintptr_t)CFArrayGetValueAtIndex(tableTags.get(), index);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to