Changeset 82923 in webkit

Timestamp:

Apr 5, 2011 4:08:45 AM (13 years ago)

Author:

podivilov@chromium.org

Message:

2011-04-05 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r82915.
​http://trac.webkit.org/changeset/82915
​https://bugs.webkit.org/show_bug.cgi?id=57825

Broke 270 chromium tests on win and linux (Requested by
podivilov on #webkit).

• rendering/RenderThemeChromiumSkia.cpp: (WebCore::RenderThemeChromiumSkia::defaultGUIFont):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderThemeChromiumSkia.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-04-05  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r82915.
+        http://trac.webkit.org/changeset/82915
+        https://bugs.webkit.org/show_bug.cgi?id=57825
+
+        Broke 270 chromium tests on win and linux (Requested by
+        podivilov on #webkit).
+
+        * rendering/RenderThemeChromiumSkia.cpp:
+        (WebCore::RenderThemeChromiumSkia::defaultGUIFont):
+
 2011-04-05  James Simonsen  <simonjam@chromium.org>
 

trunk/Source/WebCore/rendering/RenderThemeChromiumSkia.cpp

@@ -77 +77 @@
 // 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. For other non-latin
-// scripts (e.g. CJK) that Arial does not cover, we specify sans-serif as a
-// fallback.
+// sizes (e.g. 15px). So, for now we just use Arial.
 const String& RenderThemeChromiumSkia::defaultGUIFont()
 {
-    DEFINE_STATIC_LOCAL(String, fontFace, ("Arial, sans-serif"));
+    DEFINE_STATIC_LOCAL(String, fontFace, ("Arial"));
     return fontFace;
 }