Title: [185450] trunk/Source/WebCore
- Revision
- 185450
- Author
- [email protected]
- Date
- 2015-06-11 00:00:17 -0700 (Thu, 11 Jun 2015)
Log Message
[iOS] Tahoma isn't installed on iOS, so there is no need to blacklist it
https://bugs.webkit.org/show_bug.cgi?id=145865
Reviewed by Dan Bernstein.
No new tests because I can't copy Tahoma into our open source repository.
* platform/graphics/cocoa/FontCocoa.mm:
(WebCore::fontFamilyShouldNotBeUsedForArabic):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (185449 => 185450)
--- trunk/Source/WebCore/ChangeLog 2015-06-11 06:55:48 UTC (rev 185449)
+++ trunk/Source/WebCore/ChangeLog 2015-06-11 07:00:17 UTC (rev 185450)
@@ -1,3 +1,15 @@
+2015-06-11 Myles C. Maxfield <[email protected]>
+
+ [iOS] Tahoma isn't installed on iOS, so there is no need to blacklist it
+ https://bugs.webkit.org/show_bug.cgi?id=145865
+
+ Reviewed by Dan Bernstein.
+
+ No new tests because I can't copy Tahoma into our open source repository.
+
+ * platform/graphics/cocoa/FontCocoa.mm:
+ (WebCore::fontFamilyShouldNotBeUsedForArabic):
+
2015-06-10 Commit Queue <[email protected]>
Unreviewed, rolling out r185442.
Modified: trunk/Source/WebCore/platform/graphics/cocoa/FontCocoa.mm (185449 => 185450)
--- trunk/Source/WebCore/platform/graphics/cocoa/FontCocoa.mm 2015-06-11 06:55:48 UTC (rev 185449)
+++ trunk/Source/WebCore/platform/graphics/cocoa/FontCocoa.mm 2015-06-11 07:00:17 UTC (rev 185450)
@@ -84,12 +84,10 @@
if (!fontFamilyName)
return false;
- // Times New Roman contains Arabic glyphs, but Core Text doesn't know how to shape them. <rdar://problem/9823975>
+ // Times New Roman and Arial are not performant enough to use. <rdar://problem/21333326>
// FIXME <rdar://problem/12096835> remove this function once the above bug is fixed.
- // Arial and Tahoma are have performance issues so don't use them as well.
return (CFStringCompare(CFSTR("Times New Roman"), fontFamilyName, 0) == kCFCompareEqualTo)
- || (CFStringCompare(CFSTR("Arial"), fontFamilyName, 0) == kCFCompareEqualTo)
- || (CFStringCompare(CFSTR("Tahoma"), fontFamilyName, 0) == kCFCompareEqualTo);
+ || (CFStringCompare(CFSTR("Arial"), fontFamilyName, 0) == kCFCompareEqualTo);
}
#endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
