Changeset 167043 in webkit

Timestamp:

Apr 9, 2014 3:33:16 PM (10 years ago)

Author:

mmaxfield@apple.com

Message:

systemHiddenFontFamilySet in DRT is unnecessary
​https://bugs.webkit.org/show_bug.cgi?id=131414

Reviewed by Tim Horton.

It doesn't seem to affect test results on my machine.

• DumpRenderTree/mac/DumpRenderTree.mm:

(drt_NSFontManager_availableFonts):
(systemHiddenFontFamilySet): Deleted.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/mac/DumpRenderTree.mm (modified) (2 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2014-04-09  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        systemHiddenFontFamilySet in DRT is unnecessary
+        https://bugs.webkit.org/show_bug.cgi?id=131414
+
+        Reviewed by Tim Horton.
+
+        It doesn't seem to affect test results on my machine.
+
+        * DumpRenderTree/mac/DumpRenderTree.mm:
+        (drt_NSFontManager_availableFonts):
+        (systemHiddenFontFamilySet): Deleted.
+
 2014-04-09  Brent Fulgham  <bfulgham@apple.com>
 

trunk/Tools/DumpRenderTree/mac/DumpRenderTree.mm

@@ -417 +417 @@
 }
 
-static NSSet *systemHiddenFontFamilySet()
-{
-    static NSSet *fontFamilySet = [[NSSet setWithObjects:
-        @".LucidaGrandeUI",
-        nil] retain];
-
-    return fontFamilySet;
-}
-
 static IMP appKitAvailableFontFamiliesIMP;
 static IMP appKitAvailableFontsIMP;
@@ -458 +449 @@
             [availableFontList addObject:[fontInfo objectAtIndex:0]];
         }
-    }
-
-    for (NSString *hiddenFontFamily in systemHiddenFontFamilySet()) {
-        [availableFontList addObject:hiddenFontFamily];
     }