commit 733e4cfb8b3cae83daff5cc7a426ee901227dbc6
author Anders Edenbrandt <anders.edenbrandt@sonyericsson.com> Wed Feb 10 10:41:22 2010 +0100
committer Johan Redestig <johan.redestig@sonyericsson.com> Wed Apr 21 08:55:37 2010 +0200
tree 52c33b733b620fa8e2de9448791b10c2ee9b58bf
parent 18342a41ab72e2c21931afaaab6f1b9bdbedb9fa

Fix focus ring on links with non-latin text

When a clickable link in a HTML document is focused, a rectangle with
rounded corners is drawn around that link. If the link contains Arabic
characters (or characters from other non-latin alphabets) the focus
ring will be drawn too high up on the page.

Change-Id: Ia589236c4e315af76c6e8f746cc99af63e7e53d7

WebCore/platform/graphics/android/FontAndroid.cpp

diff --git a/WebCore/platform/graphics/android/FontAndroid.cpp b/WebCore/platform/graphics/android/FontAndroid.cpp
index a430b07..e9cf4c8 100644
--- a/WebCore/platform/graphics/android/FontAndroid.cpp
+++ b/WebCore/platform/graphics/android/FontAndroid.cpp
@@ -188,7 +188,7 @@
     SkScalar spacing = paint.getFontMetrics(&metrics);
     
     return FloatRect(point.x(),
-					 point.y() - floorf(SkScalarToFloat(-metrics.fAscent)),
+                     point.y(),
                      roundf(SkScalarToFloat(width)),
                      roundf(SkScalarToFloat(spacing)));
 }