2009-02-19 Jungshik Shin <jshin@chromium.org>
authordglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Feb 2009 20:12:58 +0000 (20:12 +0000)
committerdglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Feb 2009 20:12:58 +0000 (20:12 +0000)
        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=20531
        Chromium-part follow-up to the patch landed in r40636
        for bug 20531.
        Remove a static member function alternateFamilyName
        from Chromium's Win/Linux ports of FontCache. In r40636,
        alternateFamilyName was customized using #ifdef PLATFORM(WIN_OS).
        So, there's no more need for the static member function per
        platform.

        * platform/graphics/chromium/FontCacheChromiumWin.cpp:
        * platform/graphics/chromium/FontCacheLinux.cpp:

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

WebCore/ChangeLog
WebCore/platform/graphics/chromium/FontCacheChromiumWin.cpp
WebCore/platform/graphics/chromium/FontCacheLinux.cpp

index 0cf986e..fb5279d 100644 (file)
@@ -1,3 +1,19 @@
+2009-02-19  Jungshik Shin <jshin@chromium.org>
+
+        Reviewed by Eric Seidel.
+
+        https://bugs.webkit.org/show_bug.cgi?id=20531
+        Chromium-part follow-up to the patch landed in r40636
+        for bug 20531.
+        Remove a static member function alternateFamilyName
+        from Chromium's Win/Linux ports of FontCache. In r40636,
+        alternateFamilyName was customized using #ifdef PLATFORM(WIN_OS).
+        So, there's no more need for the static member function per
+        platform.
+
+        * platform/graphics/chromium/FontCacheChromiumWin.cpp:
+        * platform/graphics/chromium/FontCacheLinux.cpp:
+
 2009-02-19  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Adele Peterson.
index 03583a0..608c0f4 100644 (file)
@@ -401,46 +401,6 @@ const SimpleFontData* FontCache::getFontDataForCharacters(const Font& font, cons
 
 }
 
-const AtomicString& FontCache::alternateFamilyName(const AtomicString& familyName)
-{
-    // Note that mapping to Courier is removed because 
-    // because it's a bitmap font on Windows.
-    // Alias Courier -> Courier New
-    static AtomicString courier("Courier"), courierNew("Courier New");
-    if (equalIgnoringCase(familyName, courier))
-        return courierNew;
-
-    // Alias Times <-> Times New Roman.
-    static AtomicString times("Times"), timesNewRoman("Times New Roman");
-    if (equalIgnoringCase(familyName, times))
-        return timesNewRoman;
-    if (equalIgnoringCase(familyName, timesNewRoman))
-        return times;
-    
-    // Alias Helvetica <-> Arial
-    static AtomicString arial("Arial"), helvetica("Helvetica");
-    if (equalIgnoringCase(familyName, helvetica))
-        return arial;
-    if (equalIgnoringCase(familyName, arial))
-        return helvetica;
-
-    // We block bitmap fonts altogether so that we have to 
-    // alias MS Sans Serif (bitmap font) -> Microsoft Sans Serif (truetype font)
-    static AtomicString msSans("MS Sans Serif");
-    static AtomicString microsoftSans("Microsoft Sans Serif");
-    if (equalIgnoringCase(familyName, msSans))
-        return microsoftSans;
-
-    // Alias MS Serif (bitmap) -> Times New Roman (truetype font). There's no 
-    // 'Microsoft Sans Serif-equivalent' for Serif. 
-    static AtomicString msSerif("MS Serif");
-    if (equalIgnoringCase(familyName, msSerif))
-        return timesNewRoman;
-
-    // FIXME: should we map 'system' to something ('Tahoma') ? 
-    return emptyAtom;
-}
-
 FontPlatformData* FontCache::getSimilarFontPlatformData(const Font& font)
 {
     return 0;
index f187c55..89433e1 100644 (file)
@@ -91,16 +91,6 @@ const SimpleFontData* FontCache::getFontDataForCharacters(const Font& font,
     return ret;
 }
 
-const AtomicString& FontCache::alternateFamilyName(const AtomicString& familyName)
-{
-    notImplemented();
-
-    // This is just to stop GCC emitting a warning about returning a reference
-    // to a temporary variable
-    static AtomicString a;
-    return a;
-}
-
 FontPlatformData* FontCache::getSimilarFontPlatformData(const Font& font)
 {
     return 0;