Revert "Don't extend the arrow asset of the combo box over the text, so that the real background"

This reverts commit 02b5ebb30fc88967b843818cbc61987f9dc9685d.

2 files changed, 14 insertions, 5 deletions

diff --git a/WebCore/platform/android/RenderThemeAndroid.cpp b/WebCore/platform/android/RenderThemeAndroid.cpp
index 51f404e..0f419f5 100644
--- a/WebCore/platform/android/RenderThemeAndroid.cpp
+++ b/WebCore/platform/android/RenderThemeAndroid.cpp
@@ -235,7 +235,7 @@ bool RenderThemeAndroid::paintTextArea(RenderObject* obj, const RenderObject::Pa
 {
     if (obj->isMenuList())
         return paintCombo(obj, info, rect);
-    return true;
+    return true;    
 }
 
 void RenderThemeAndroid::adjustSearchFieldStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
@@ -245,7 +245,7 @@ void RenderThemeAndroid::adjustSearchFieldStyle(CSSStyleSelector*, RenderStyle*
 
 bool RenderThemeAndroid::paintSearchField(RenderObject*, const RenderObject::PaintInfo&, const IntRect&)
 {
-    return true;
+    return true;    
 }
 
 void RenderThemeAndroid::adjustListboxStyle(CSSStyleSelector*, RenderStyle* style, Element*) const
@@ -259,6 +259,12 @@ static void adjustMenuListStyleCommon(RenderStyle* style, Element* e)
 {
     // Added to make room for our arrow.
     style->setPaddingRight(Length(RenderSkinCombo::extraWidth(), Fixed));
+    // Code copied from RenderThemeMac.mm
+    // Makes sure that the text shows up on our treatment
+    bool isEnabled = true;
+    if (e)
+        isEnabled = e->isEnabledFormControl();
+    style->setColor(isEnabled ? Color::black : Color::darkGray);
 }
 
 void RenderThemeAndroid::adjustMenuListStyle(CSSStyleSelector*, RenderStyle* style, Element* e) const
diff --git a/WebKit/android/RenderSkinCombo.cpp b/WebKit/android/RenderSkinCombo.cpp
index fd6f1f2..870c13e 100644
--- a/WebKit/android/RenderSkinCombo.cpp
+++ b/WebKit/android/RenderSkinCombo.cpp
@@ -36,9 +36,9 @@ namespace WebCore {
 
 static SkBitmap         s_bitmap[2];    // Collection of assets for a combo box
 static bool             s_decoded;      // True if all assets were decoded
-static const int        s_margin = 0;
+static const int        s_margin = 2;
 static const SkIRect    s_mar = { s_margin, s_margin,
-                                  RenderSkinCombo::extraWidth() - 2, s_margin };
+                                RenderSkinCombo::extraWidth(), s_margin };
     
 RenderSkinCombo::RenderSkinCombo()
 {
@@ -61,8 +61,11 @@ bool RenderSkinCombo::Draw(SkCanvas* canvas, Node* element, int x, int y, int wi
         return true;
 
     State state = (element->isElementNode() && static_cast<Element*>(element)->isEnabledFormControl()) ? kNormal : kDisabled;
+    if (height < (s_margin<<1) + 1) {
+        height = (s_margin<<1) + 1;
+    }
     SkRect bounds;
-    bounds.set(SkIntToScalar(x + width), SkIntToScalar(y), SkIntToScalar(x + width), SkIntToScalar(y + height));
+    bounds.set(SkIntToScalar(x), SkIntToScalar(y), SkIntToScalar(x + width), SkIntToScalar(y + height));
     SkNinePatch::DrawNine(canvas, bounds, s_bitmap[state], s_mar);
     return false;
 }