2011-04-05 Takayoshi Kochi <kochi@chromium.org>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Apr 2011 09:17:53 +0000 (09:17 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 5 Apr 2011 09:17:53 +0000 (09:17 +0000)
        Reviewed by Tony Chang.

        [chromium] Specify 'sans-serif' as a fallback to the default UI font
        'Arial' for non-latin UI scripts (e.g. CJK).
        https://bugs.webkit.org/show_bug.cgi?id=55035

        No new tests, because it may still fail if the system is not configured
        properly to resolve 'sans-serif' to existing real font file, thus even
        manual test is hard to systematically tell it is working properly.

        * rendering/RenderThemeChromiumSkia.cpp

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderThemeChromiumSkia.cpp

index 01f90b0..dadfecb 100644 (file)
@@ -1,3 +1,17 @@
+2011-04-05  Takayoshi Kochi <kochi@chromium.org>
+
+        Reviewed by Tony Chang.
+
+        [chromium] Specify 'sans-serif' as a fallback to the default UI font
+        'Arial' for non-latin UI scripts (e.g. CJK).
+        https://bugs.webkit.org/show_bug.cgi?id=55035
+
+        No new tests, because it may still fail if the system is not configured
+        properly to resolve 'sans-serif' to existing real font file, thus even
+        manual test is hard to systematically tell it is working properly.
+
+        * rendering/RenderThemeChromiumSkia.cpp
+
 2011-04-05  Kent Tamura  <tkent@chromium.org>
 
         Reviewed by Dimitri Glazkov.
index 9691876..aff0df6 100644 (file)
@@ -76,10 +76,12 @@ static const float defaultSearchFieldResultsButtonWidth = 18;
 //
 // FIXME: The only case where we know we don't match IE is for ANSI encodings.
 // IE uses MS Shell Dlg there, which we render incorrectly at certain pixel
-// sizes (e.g. 15px). So, for now we just use Arial.
+// sizes (e.g. 15px). So, for now we just use Arial. For other non-latin
+// scripts (e.g. CJK) that Arial does not cover, we specify sans-serif as a
+// fallback.
 const String& RenderThemeChromiumSkia::defaultGUIFont()
 {
-    DEFINE_STATIC_LOCAL(String, fontFace, ("Arial"));
+    DEFINE_STATIC_LOCAL(String, fontFace, ("Arial, sans-serif"));
     return fontFace;
 }