Title: [183305] trunk/Source/WebCore
- Revision
- 183305
- Author
- [email protected]
- Date
- 2015-04-24 20:30:20 -0700 (Fri, 24 Apr 2015)
Log Message
[Cocoa] FontPlatformData's equality check should always use reference URLs
https://bugs.webkit.org/show_bug.cgi?id=144168
Reviewed by Tim Horton.
<rdar://problem/18985642> is not fixed, so we need to continue using the older
objectForEqualityCheck().
No new tests because there is no behavior change.
* platform/graphics/cocoa/FontPlatformDataCocoa.mm:
(WebCore::FontPlatformData::objectForEqualityCheck):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (183304 => 183305)
--- trunk/Source/WebCore/ChangeLog 2015-04-25 03:29:23 UTC (rev 183304)
+++ trunk/Source/WebCore/ChangeLog 2015-04-25 03:30:20 UTC (rev 183305)
@@ -1,5 +1,20 @@
2015-04-24 Myles C. Maxfield <[email protected]>
+ [Cocoa] FontPlatformData's equality check should always use reference URLs
+ https://bugs.webkit.org/show_bug.cgi?id=144168
+
+ Reviewed by Tim Horton.
+
+ <rdar://problem/18985642> is not fixed, so we need to continue using the older
+ objectForEqualityCheck().
+
+ No new tests because there is no behavior change.
+
+ * platform/graphics/cocoa/FontPlatformDataCocoa.mm:
+ (WebCore::FontPlatformData::objectForEqualityCheck):
+
+2015-04-24 Myles C. Maxfield <[email protected]>
+
Implement parsing support for font-synthesis CSS property
https://bugs.webkit.org/show_bug.cgi?id=144180
<rdar://problem/20692791>
Modified: trunk/Source/WebCore/platform/graphics/cocoa/FontPlatformDataCocoa.mm (183304 => 183305)
--- trunk/Source/WebCore/platform/graphics/cocoa/FontPlatformDataCocoa.mm 2015-04-25 03:29:23 UTC (rev 183304)
+++ trunk/Source/WebCore/platform/graphics/cocoa/FontPlatformDataCocoa.mm 2015-04-25 03:30:20 UTC (rev 183305)
@@ -268,16 +268,12 @@
RetainPtr<CFTypeRef> FontPlatformData::objectForEqualityCheck(CTFontRef ctFont)
{
-#if (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED <= 80000) || (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED <= 101000)
auto fontDescriptor = adoptCF(CTFontCopyFontDescriptor(ctFont));
// FIXME: https://bugs.webkit.org/show_bug.cgi?id=138683 This is a shallow pointer compare for web fonts
// because the URL contains the address of the font. This means we might erroneously get false negatives.
RetainPtr<CFURLRef> url = "" kCTFontReferenceURLAttribute)));
ASSERT(CFGetTypeID(url.get()) == CFURLGetTypeID());
return url;
-#else
- return adoptCF(CTFontCopyGraphicsFont(ctFont, 0));
-#endif
}
RetainPtr<CFTypeRef> FontPlatformData::objectForEqualityCheck() const
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes