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

git-svn-id: svn://svn.chromium.org/blink/trunk@82915 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 18bb4f21
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.
......@@ -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;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment