Unreviewed, rolling out r167713.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Apr 2014 18:36:26 +0000 (18:36 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Apr 2014 18:36:26 +0000 (18:36 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132070

broke hundreds of tests (Requested by thorton on #webkit).

Reverted changeset:

"[OS X] Make checking if a font is the system font more
robust"
https://bugs.webkit.org/show_bug.cgi?id=132030
http://trac.webkit.org/changeset/167713

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/Font.cpp
Source/WebCore/platform/graphics/Font.h
Source/WebCore/platform/graphics/mac/FontMac.mm

index 19c298a..bb9e52c 100644 (file)
@@ -1,3 +1,17 @@
+2014-04-23  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r167713.
+        https://bugs.webkit.org/show_bug.cgi?id=132070
+
+        broke hundreds of tests (Requested by thorton on #webkit).
+
+        Reverted changeset:
+
+        "[OS X] Make checking if a font is the system font more
+        robust"
+        https://bugs.webkit.org/show_bug.cgi?id=132030
+        http://trac.webkit.org/changeset/167713
+
 2014-04-22  David Hyatt  <hyatt@apple.com>
 
         [New Multicolumn] Nested columns not working at all.
index 6ce31b2..f276f13 100644 (file)
@@ -481,11 +481,11 @@ bool Font::hasValidAverageCharWidth() const
     if (family.isEmpty())
         return false;
 
-#if PLATFORM(MAC) || PLATFORM(IOS)
-    // Internal fonts on OS X and iOS also have an invalid entry in the table for avgCharWidth.
-    if (primaryFontDataIsSystemFont())
+    // Internal fonts on OS X also have an invalid entry in the table for avgCharWidth.
+    // They are hidden by having a name that begins with a period, so simply search
+    // for that here rather than try to keep the list up to date.
+    if (family.startsWith('.'))
         return false;
-#endif
 
     static HashSet<AtomicString>* fontFamiliesWithInvalidCharWidthMap = 0;
 
index 63525c9..a35b9bb 100644 (file)
@@ -188,8 +188,6 @@ public:
     CodePath codePath(const TextRun&) const;
     static CodePath characterRangeCodePath(const LChar*, unsigned) { return Simple; }
     static CodePath characterRangeCodePath(const UChar*, unsigned len);
-    
-    bool primaryFontDataIsSystemFont() const;
 
 private:
     enum ForTextEmphasisOrNot { NotForTextEmphasis, ForTextEmphasis };
index fa70d64..d9300b5 100644 (file)
 #endif
 #import <wtf/MathExtras.h>
 
-#if __has_include(<CoreText/CTFontDescriptorPriv.h>)
-#import <CoreText/CTFontDescriptorPriv.h>
-#endif
-extern "C" bool CTFontDescriptorIsSystemUIFont(CTFontDescriptorRef);
-
 #if ENABLE(LETTERPRESS)
 #import "SoftLinking.h"
 #if __has_include(<CoreGraphics/CoreGraphicsPrivate.h>)
@@ -540,14 +535,4 @@ DashArray Font::dashesForIntersectionsWithRect(const TextRun& run, const FloatPo
 }
 #endif
 
-bool Font::primaryFontDataIsSystemFont() const
-{
-#if __MAC_OS_X_VERSION_MIN_REQUIRED >= 1090
-    RetainPtr<CTFontDescriptorRef> descriptor = CTFontCopyFontDescriptor(primaryFont()->platformData().ctFont());
-    return CTFontDescriptorIsSystemUIFont(descriptor.get());
-#else
-    return false;
-#endif
-}
-
 }