Cleanup RenderThemeWin.

src/webkit/port/rendering/RenderThemeWin.cpp

 /*
  * This file is part of the WebKit project.
  *
  * Copyright (C) 2006 Apple Computer, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 23 matching lines @@
 #include "FontUtilsWin.h"
 #include "GraphicsContext.h"
 #include "ScrollbarTheme.h"
 #include "SkiaUtils.h"
 #include "ThemeHelperWin.h"
 
 #include "base/gfx/native_theme.h"
 #include "base/gfx/skia_utils.h"
 #include "base/win_util.h"
 
+namespace {
+
 // These enums correspond to similarly named values defined by SafariTheme.h
 enum ControlSize {
     RegularControlSize,
     SmallControlSize,
     MiniControlSize
 };
 
 enum PaddingType {
     TopPadding,
     RightPadding,
     BottomPadding,
     LeftPadding
 };
 
-namespace {
-    const int kDefaultButtonPadding = 2;
+const int kDefaultButtonPadding = 2;
 
-    // These magic numbers come from Apple's version of RenderThemeWin.cpp.
-    const int kMenuListPadding[4] = { 1, 2, 1, 2 };
+// These magic numbers come from Apple's version of RenderThemeWin.cpp.
+const int kMenuListPadding[4] = { 1, 2, 1, 2 };
 
-    // The kLayoutTest* constants are metrics used only in layout test mode,
-    // so as to match RenderThemeMac.mm and to remain consistent despite any
-    //  theme or font changes.
-    const int kLayoutTestControlHeight[3] = { 21, 18, 15 };
-    const int kLayoutTestButtonPadding[4] = { 0, 8, 0, 8 };
-    const int kLayoutTestStyledMenuListInternalPadding[4] = { 1, 0, 2, 8 };
-    const int kLayoutTestMenuListInternalPadding[3][4] =
-    {
-        { 2, 26, 3, 8 },
-        { 2, 23, 3, 8 },
-        { 2, 22, 3, 10 }
-    };
-    const int kLayoutTestMenuListMinimumWidth[3] = { 9, 5, 0 };
-    const float kLayoutTestBaseFontSize = 11.0f;
-    const float kLayoutTestStatusBarFontSize = 10.0f;
-    const float kLayoutTestSystemFontSize = 13.0f;
+// The kLayoutTest* constants are metrics used only in layout test mode,
+// so as to match RenderThemeMac.mm and to remain consistent despite any
+//  theme or font changes.
+const int kLayoutTestControlHeight[3] = { 21, 18, 15 };
+const int kLayoutTestButtonPadding[4] = { 0, 8, 0, 8 };
+const int kLayoutTestStyledMenuListInternalPadding[4] = { 1, 0, 2, 8 };
+const int kLayoutTestMenuListInternalPadding[3][4] =
+{
+    { 2, 26, 3, 8 },
+    { 2, 23, 3, 8 },
+    { 2, 22, 3, 10 }
+};
+const int kLayoutTestMenuListMinimumWidth[3] = { 9, 5, 0 };
+const float kLayoutTestBaseFontSize = 11.0f;
+const float kLayoutTestStatusBarFontSize = 10.0f;
+const float kLayoutTestSystemFontSize = 13.0f;
 
-    const int kLayoutTestSliderThumbWidth = 15;
-    const int kLayoutTestSliderThumbHeight = 15;
+const int kLayoutTestSliderThumbWidth = 15;
+const int kLayoutTestSliderThumbHeight = 15;
 
-    const int kLayoutTestMenuListButtonWidth = 15;
-    const int kLayoutTestButtonMinHeight = 15;
+const int kLayoutTestMenuListButtonWidth = 15;
+const int kLayoutTestButtonMinHeight = 15;
 
-    const int kLayoutTestSearchFieldHeight[3] = { 22, 19, 17 };
-    const int kLayoutTestEmptyResultsOffset = 9;
-    const int kLayoutTestResultsArrowWidth = 5;
+const int kLayoutTestSearchFieldHeight[3] = { 22, 19, 17 };
+const int kLayoutTestEmptyResultsOffset = 9;
+const int kLayoutTestResultsArrowWidth = 5;
 
-    const short kLayoutTestSearchFieldBorderWidth = 2;
-    const int kLayoutTestSearchFieldPadding = 1;
+const short kLayoutTestSearchFieldBorderWidth = 2;
+const int kLayoutTestSearchFieldPadding = 1;
 
+// Constants that are used in non-layout-test mode.
+const int kStyledMenuListInternalPadding[4] = { 1, 4, 1, 4 };
 
+// The default variable-width font size.  We use this as the default font
+// size for the "system font", and as a base size (which we then shrink) for
+// form control fonts.
+float DefaultFontSize = 16.0;
 
-    // Constants that are used in non-layout-test mode.
-    const int kStyledMenuListInternalPadding[4] = { 1, 4, 1, 4 };
+WebCore::FontDescription smallSystemFont;
+WebCore::FontDescription menuFont;
+WebCore::FontDescription labelFont;
 
-    // The default variable-width font size.  We use this as the default font
-    // size for the "system font", and as a base size (which we then shrink) for
-    // form control fonts.
-    float DefaultFontSize = 16.0;
+}  // namespace
 
-    WebCore::FontDescription smallSystemFont;
-    WebCore::FontDescription menuFont;
-    WebCore::FontDescription labelFont;
+namespace WebCore {
+
+// Internal static helper functions.  We don't put them in an anonymous
+// namespace so they have easier access to the WebCore namespace.
+
+static bool supportsFocus(ControlPart appearance)
+{
+    switch (appearance) {
+        case PushButtonPart:
+        case ButtonPart:
+        case DefaultButtonPart:
+        case TextFieldPart:
+        case TextAreaPart:
+            return true;
+        default:
+            return false;
+    }
+
+    return false;
 }
 
-namespace WebCore {
-  
 static void setFixedPadding(RenderStyle* style, const int padding[4])
 {
     style->setPaddingLeft(Length(padding[LeftPadding], Fixed));
     style->setPaddingRight(Length(padding[RightPadding], Fixed));
     style->setPaddingTop(Length(padding[TopPadding], Fixed));
     style->setPaddingBottom(Length(padding[BottomPadding], Fixed));
 }
 
 // This is logic from RenderThemeMac.mm, and is used by layout test mode.
 static ControlSize controlSizeForFont(RenderStyle* style)
 {
     if (style->fontSize() >= 16) {
         return RegularControlSize;
     } else if (style->fontSize() >= 11) {
         return SmallControlSize;
     }
     return MiniControlSize;
 }
 
-RenderTheme* theme()
-{
-    static RenderThemeWin winTheme;
-    return &winTheme;
-}
-
-RenderThemeWin::RenderThemeWin()
-{
-}
-
-RenderThemeWin::~RenderThemeWin()
-{
-}
-
-Color RenderThemeWin::platformActiveSelectionBackgroundColor() const
-{
-    if (ChromiumBridge::layoutTestMode())
-        return Color("#0000FF");  // Royal blue
-    COLORREF color = GetSysColor(COLOR_HIGHLIGHT);
-    return Color(GetRValue(color), GetGValue(color), GetBValue(color), 255);
-}
-
-Color RenderThemeWin::platformInactiveSelectionBackgroundColor() const
-{
-    if (ChromiumBridge::layoutTestMode())
-        return Color("#999999");  // Medium grey
-    COLORREF color = GetSysColor(COLOR_GRAYTEXT);
-    return Color(GetRValue(color), GetGValue(color), GetBValue(color), 255);
-}
-
-Color RenderThemeWin::platformActiveSelectionForegroundColor() const
-{
-    if (ChromiumBridge::layoutTestMode())
-        return Color("#FFFFCC");  // Pale yellow
-    COLORREF color = GetSysColor(COLOR_HIGHLIGHTTEXT);
-    return Color(GetRValue(color), GetGValue(color), GetBValue(color), 255);
-}
-
-Color RenderThemeWin::platformInactiveSelectionForegroundColor() const
-{
-    return Color::white;
-}
-
 static float systemFontSizeForControlSize(ControlSize controlSize)
 {
     static float sizes[] = { 13.0f, 11.0f, 9.0f };
-    
+
     return sizes[controlSize];
 }
 
 // This is basically RenderThemeMac::setFontFromControlSize
 static int layoutTestSetFontFromControlSize(CSSStyleSelector* selector, RenderStyle* style)
 {
     FontDescription fontDescription;
     fontDescription.setIsAbsoluteSize(true);
     fontDescription.setGenericFamily(FontDescription::SerifFamily);
 
     float fontSize = systemFontSizeForControlSize(controlSizeForFont(style));
     fontDescription.firstFamily().setFamily("Lucida Grande");
 
     fontDescription.setComputedSize(fontSize);
     fontDescription.setSpecifiedSize(fontSize);
-    
+
     // Reset line height
     style->setLineHeight(RenderStyle::initialLineHeight());
 
     style->setFontDescription(fontDescription);
         style->font().update(0);
 
     return 0;
 }
 
 // Return the height of system font |font| in pixels.  We use this size by
@@ skipping 47 matching lines @@
     // once GetFontFamilyForScript is enhanced to support GenericFamilyType for
     // real. For now, we make sure that we use Arial to match IE for those
     // scripts.
     if (dominantScript != USCRIPT_LATIN &&
         dominantScript != USCRIPT_CYRILLIC &&
         dominantScript != USCRIPT_GREEK &&
         dominantScript != USCRIPT_INVALID_CODE) {
         family = GetFontFamilyForScript(dominantScript, GENERIC_FAMILY_NONE);
         if (family)
             return const_cast<wchar_t*>(family);
-    } 
+    }
     return L"Arial";
 }
 
 // Converts |points| to pixels.  One point is 1/72 of an inch.
 static float pointsToPixels(float points)
 {
     static float pixelsPerInch = 0.0f;
     if (!pixelsPerInch) {
         HDC hdc = GetDC(0);  // What about printing?  Is this the right DC?
         if (hdc) {  // Can this ever actually be NULL?
             pixelsPerInch = GetDeviceCaps(hdc, LOGPIXELSY);
             ReleaseDC(0, hdc);
         } else {
             pixelsPerInch = 96.0f;
         }
     }
 
     static const float POINTS_PER_INCH = 72.0f;
     return points / POINTS_PER_INCH * pixelsPerInch;
 }
 
+static void setSizeIfAuto(RenderStyle* style, const IntSize& size)
+{
+    if (style->width().isIntrinsicOrAuto())
+        style->setWidth(Length(size.width(), Fixed));
+    if (style->height().isAuto())
+        style->setHeight(Length(size.height(), Fixed));
+}
+
+static IntSize layoutTestCheckboxSize(RenderStyle* style)
+{
+    static const IntSize sizes[3] = { IntSize(14, 14), IntSize(12, 12), IntSize(10, 10) };
+    return sizes[controlSizeForFont(style)];
+}
+
+static IntSize layoutTestRadioboxSize(RenderStyle* style)
+{
+    static const IntSize sizes[3] = { IntSize(14, 15), IntSize(12, 13), IntSize(10, 10) };
+    return sizes[controlSizeForFont(style)];
+}
+
+// Hacks for using Mac menu list metrics when in layout test mode.
+static int layoutTestMenuListInternalPadding(RenderStyle* style, int paddingType)
+{
+    if (style->appearance() == MenulistPart) {
+        return kLayoutTestMenuListInternalPadding[controlSizeForFont(style)][paddingType];
+    }
+    if (style->appearance() == MenulistButtonPart) {
+        if (paddingType == RightPadding) {
+            const float baseArrowWidth = 5.0f;
+            float fontScale = style->fontSize() / kLayoutTestBaseFontSize;
+            float arrowWidth = ceilf(baseArrowWidth * fontScale);
+
+            int arrowPaddingLeft = 6;
+            int arrowPaddingRight = 6;
+            int paddingBeforeSeparator = 4;
+            // Add 2 for separator space, seems to match RenderThemeMac::paintMenuListButton.
+            return static_cast<int>(arrowWidth + arrowPaddingLeft + arrowPaddingRight +
+                                    paddingBeforeSeparator);
+        } else {
+            return kLayoutTestStyledMenuListInternalPadding[paddingType];
+        }
+    }
+    return 0;
+}
+
+static const IntSize* layoutTestCancelButtonSizes()
+{
+    static const IntSize sizes[3] = { IntSize(16, 13), IntSize(13, 11), IntSize(13, 9) };
+    return sizes;
+}
+
+static const IntSize* layoutTestResultsButtonSizes()
+{
+    static const IntSize sizes[3] = { IntSize(19, 13), IntSize(17, 11), IntSize(17, 9) };
+    return sizes;
+}
+
+// Implement WebCore::theme() for getting the global RenderTheme.
+RenderTheme* theme()
+{
+    static RenderThemeWin winTheme;
+    return &winTheme;
+}
+
+bool RenderThemeWin::supportsFocusRing(const RenderStyle* style) const
+{
+   // Let webkit draw one of its halo rings around any focused element,
+   // except push buttons. For buttons we use the windows PBS_DEFAULTED
+   // styling to give it a blue border.
+   return style->appearance() == ButtonPart ||
+          style->appearance() == PushButtonPart;
+}
+
+Color RenderThemeWin::platformActiveSelectionBackgroundColor() const
+{
+    if (ChromiumBridge::layoutTestMode())
+        return Color("#0000FF");  // Royal blue
+    COLORREF color = GetSysColor(COLOR_HIGHLIGHT);
+    return Color(GetRValue(color), GetGValue(color), GetBValue(color), 255);
+}
+
+Color RenderThemeWin::platformInactiveSelectionBackgroundColor() const
+{
+    if (ChromiumBridge::layoutTestMode())
+        return Color("#999999");  // Medium grey
+    COLORREF color = GetSysColor(COLOR_GRAYTEXT);
+    return Color(GetRValue(color), GetGValue(color), GetBValue(color), 255);
+}
+
+Color RenderThemeWin::platformActiveSelectionForegroundColor() const
+{
+    if (ChromiumBridge::layoutTestMode())
+        return Color("#FFFFCC");  // Pale yellow
+    COLORREF color = GetSysColor(COLOR_HIGHLIGHTTEXT);
+    return Color(GetRValue(color), GetGValue(color), GetBValue(color), 255);
+}
+
+Color RenderThemeWin::platformInactiveSelectionForegroundColor() const
+{
+    return Color::white;
+}
+
 double RenderThemeWin::caretBlinkFrequency() const
 {
     // Disable the blinking caret in layout test mode, as it introduces
     // a race condition for the pixel tests. http://b/1198440
     if (ChromiumBridge::layoutTestMode())
         return 0;
 
     // TODO(ericroman): this should be using the platform's blink frequency.
     return RenderTheme::caretBlinkFrequency();
 }
@@ skipping 92 matching lines @@
         }
         cachedDesc->setIsAbsoluteSize(true);
         cachedDesc->setGenericFamily(FontDescription::NoFamily);
         cachedDesc->setSpecifiedSize(fontSize);
         cachedDesc->setWeight(FontWeightNormal);
         cachedDesc->setItalic(false);
     }
     fontDescription = *cachedDesc;
 }
 
-static bool supportsFocus(ControlPart appearance)
-{
-    switch (appearance) {
-        case PushButtonPart:
-        case ButtonPart:
-        case DefaultButtonPart:
-        case TextFieldPart:
-        case TextAreaPart:
-            return true;
-        default:
-            return false;
-    }
-
-    return false;
-}
-
-bool RenderThemeWin::supportsFocusRing(const RenderStyle* style) const
-{
-   // Let webkit draw one of its halo rings around any focused element,
-   // except push buttons. For buttons we use the windows PBS_DEFAULTED
-   // styling to give it a blue border.
-   return style->appearance() == ButtonPart ||
-          style->appearance() == PushButtonPart;
-}
-
-unsigned RenderThemeWin::determineState(RenderObject* o)
-{
-    unsigned result = TS_NORMAL;
-    ControlPart appearance = o->style()->appearance();
-    if (!isEnabled(o))
-        result = TS_DISABLED;
-    else if (isReadOnlyControl(o) && (TextFieldPart == appearance || TextAreaPart == appearance))
-        result = ETS_READONLY; // Readonly is supported on textfields.
-    else if (isPressed(o)) // Active overrides hover and focused.
-        result = TS_PRESSED;
-    else if (supportsFocus(appearance) && isFocused(o))
-        result = ETS_FOCUSED;
-    else if (isHovered(o))
-        result = TS_HOT;
-    if (isChecked(o))
-        result += 4; // 4 unchecked states, 4 checked states.
-    return result;
-}
-
-unsigned RenderThemeWin::determineClassicState(RenderObject* o)
-{
-    unsigned result = 0;
-    if (!isEnabled(o))
-        result = DFCS_INACTIVE;
-    else if (isPressed(o)) // Active supersedes hover
-        result = DFCS_PUSHED;
-    else if (isHovered(o))
-        result = DFCS_HOT;
-    if (isChecked(o))
-        result |= DFCS_CHECKED;
-    return result;
-}
-
-ThemeData RenderThemeWin::getThemeData(RenderObject* o)
-{
-    ThemeData result;
-    switch (o->style()->appearance()) {
-        case PushButtonPart:
-        case ButtonPart:
-            result.m_part = BP_PUSHBUTTON;
-            result.m_classicState = DFCS_BUTTONPUSH;
-            break;
-        case CheckboxPart:
-            result.m_part = BP_CHECKBOX;
-            result.m_classicState = DFCS_BUTTONCHECK;
-            break;
-        case RadioPart:
-            result.m_part = BP_RADIOBUTTON;
-            result.m_classicState = DFCS_BUTTONRADIO;
-            break;
-        case ListboxPart:
-        case MenulistPart:
-        case TextFieldPart:
-        case TextAreaPart:
-            result.m_part = ETS_NORMAL;
-            break;
-    }
-
-    result.m_state = determineState(o);
-    result.m_classicState |= determineClassicState(o);
-
-    return result;
-}
-
-bool RenderThemeWin::paintButton(RenderObject* o,
-                                 const RenderObject::PaintInfo& i,
-                                 const IntRect& r)
-{
-    const ThemeData& themeData = getThemeData(o);
-
-    WebCore::ThemeHelperWin helper(i.context, r);
-    gfx::PlatformCanvas* canvas = helper.context()->platformContext()->canvas();
-
-    HDC hdc = canvas->beginPlatformPaint();
-    int state = themeData.m_state;
-    RECT renderRect = helper.rect();
-
-    gfx::NativeTheme::instance()->PaintButton(hdc,
-                                              themeData.m_part,
-                                              state,
-                                              themeData.m_classicState,
-                                              &renderRect);
-    canvas->endPlatformPaint();
-    return false;
-}
-
-static void setSizeIfAuto(RenderStyle* style, const IntSize& size)
-{
-    if (style->width().isIntrinsicOrAuto())
-        style->setWidth(Length(size.width(), Fixed));
-    if (style->height().isAuto())
-        style->setHeight(Length(size.height(), Fixed));
-}
-
 int RenderThemeWin::minimumMenuListSize(RenderStyle* style) const
 {
     if (ChromiumBridge::layoutTestMode()) {
         return kLayoutTestMenuListMinimumWidth[controlSizeForFont(style)];
     } else {
         return 0;
     }
 }
 
-static IntSize layoutTestCheckboxSize(RenderStyle* style)
-{
-    static const IntSize sizes[3] = { IntSize(14, 14), IntSize(12, 12), IntSize(10, 10) };
-    return sizes[controlSizeForFont(style)];
-}
-
-static IntSize layoutTestRadioboxSize(RenderStyle* style)
-{
-    static const IntSize sizes[3] = { IntSize(14, 15), IntSize(12, 13), IntSize(10, 10) };
-    return sizes[controlSizeForFont(style)];
-}
-
 void RenderThemeWin::setCheckboxSize(RenderStyle* style) const
 {
     // If the width and height are both specified, then we have nothing to do.
     if (!style->width().isIntrinsicOrAuto() && !style->height().isAuto())
         return;
 
     // FIXME:  A hard-coded size of 13 is used.  This is wrong but necessary for now.  It matches Firefox.
     // At different DPI settings on Windows, querying the theme gives you a larger size that accounts for
     // the higher DPI.  Until our entire engine honors a DPI setting other than 96, we can't rely on the theme's
     // metrics.
     const IntSize size = ChromiumBridge::layoutTestMode() ?
         layoutTestCheckboxSize(style) : IntSize(13, 13);
     setSizeIfAuto(style, size);
 }
 
 void RenderThemeWin::setRadioSize(RenderStyle* style) const
 {
     if (ChromiumBridge::layoutTestMode()) {
         setSizeIfAuto(style, layoutTestRadioboxSize(style));
     } else {
         // Use same sizing for radio box as checkbox.
         setCheckboxSize(style);
     }
 }
 
+bool RenderThemeWin::paintButton(RenderObject* o,
+                                 const RenderObject::PaintInfo& i,
+                                 const IntRect& r)
+{
+    const ThemeData& themeData = getThemeData(o);
+
+    WebCore::ThemeHelperWin helper(i.context, r);
+    gfx::PlatformCanvas* canvas = helper.context()->platformContext()->canvas();
+
+    HDC hdc = canvas->beginPlatformPaint();
+    int state = themeData.m_state;
+    RECT renderRect = helper.rect();
+
+    gfx::NativeTheme::instance()->PaintButton(hdc,
+                                              themeData.m_part,
+                                              state,
+                                              themeData.m_classicState,
+                                              &renderRect);
+    canvas->endPlatformPaint();
+    return false;
+}
+
 bool RenderThemeWin::paintTextField(RenderObject* o, const RenderObject::PaintInfo& i, const IntRect& r)
 {
     return paintTextFieldInternal(o, i, r, true);
 }
 
-bool RenderThemeWin::paintTextFieldInternal(RenderObject* o,
-                                            const RenderObject::PaintInfo& i,
-                                            const IntRect& r,
-                                            bool drawEdges)
-{
-    // Nasty hack to make us not paint the border on text fields with a
-    // border-radius. Webkit paints elements with border-radius for us.
-    // TODO(ojan): Get rid of this if-check once we can properly clip rounded
-    // borders: http://b/1112604 and http://b/1108635
-    // TODO(ojan): make sure we do the right thing if css background-clip is
-    // set.
-    if (o->style()->hasBorderRadius())
-        return false;
-
-    const ThemeData& themeData = getThemeData(o);
-
-    WebCore::ThemeHelperWin helper(i.context, r);
-    gfx::PlatformCanvas* canvas = helper.context()->platformContext()->canvas();
-
-    HDC hdc = canvas->beginPlatformPaint();
-    COLORREF clr = gfx::SkColorToCOLORREF(o->style()->backgroundColor().rgb());
-    RECT renderRect = helper.rect();
-
-    gfx::NativeTheme::instance()->PaintTextField(hdc,
-                                                 themeData.m_part,
-                                                 themeData.m_state,
-                                                 themeData.m_classicState,
-                                                 &renderRect,
-                                                 clr,
-                                                 true,
-                                                 drawEdges);
-    canvas->endPlatformPaint();
-    return false;
-}
-
 bool RenderThemeWin::paintSearchField(RenderObject* o, const RenderObject::PaintInfo& i, const IntRect& r)
 {
     return paintTextField(o, i, r);
 }
 
 void RenderThemeWin::adjustMenuListStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
 {
     // Height is locked to auto on all browsers.
     style->setLineHeight(RenderStyle::initialLineHeight());
 
     if (ChromiumBridge::layoutTestMode()) {
         style->resetBorder();
         style->setHeight(Length(Auto));
         // Select one of the 3 fixed heights for controls
         style->resetPadding();
         if (style->height().isAuto()) {
             // RenderThemeMac locks the size to 3 distinct values (NSControlSize).
             // We on the other hand, base the height off the font.
             int fixedHeight = kLayoutTestControlHeight[controlSizeForFont(style)];
             style->setHeight(Length(fixedHeight, Fixed));
         }
         layoutTestSetFontFromControlSize(selector, style);
     }
 }
 
-void RenderThemeWin::adjustMenuListButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
-{
-    adjustMenuListStyle(selector, style, e);
-}
-
 // Used to paint unstyled menulists (i.e. with the default border)
 bool RenderThemeWin::paintMenuList(RenderObject* o, const RenderObject::PaintInfo& i, const IntRect& r)
 {
     int borderRight = o->borderRight();
     int borderLeft = o->borderLeft();
     int borderTop = o->borderTop();
     int borderBottom = o->borderBottom();
 
     // If all the borders are 0, then tell skia not to paint the border on the textfield.
-    // TODO(ojan): http://b/1210017 Figure out how to get Windows to not draw individual 
+    // TODO(ojan): http://b/1210017 Figure out how to get Windows to not draw individual
     // borders and then pass that to skia so we can avoid drawing any borders that are
     // set to 0. For non-zero borders, we draw the border, but webkit just draws
     // over it.
     // TODO(ojan): layout-test-mode removes borders, so we end up never drawing
     // edges in layout-test-mode. See adjustMenuListStyle, style->resetBorder().
     // We really need to remove the layout-test-mode only hacks.
     bool drawEdges = !(borderRight == 0 && borderLeft == 0 && borderTop == 0 && borderBottom == 0);
 
     paintTextFieldInternal(o, i, r, drawEdges);
 
     // Take padding and border into account.
     // If the MenuList is smaller than the size of a button, make sure to
-    // shrink it appropriately and not put its x position to the left of 
+    // shrink it appropriately and not put its x position to the left of
     // the menulist.
     const int buttonWidth = ChromiumBridge::layoutTestMode() ?
         kLayoutTestMenuListButtonWidth : GetSystemMetrics(SM_CXVSCROLL);
     int spacingLeft = borderLeft + o->paddingLeft();
     int spacingRight = borderRight + o->paddingRight();
     int spacingTop = borderTop + o->paddingTop();
     int spacingBottom = borderBottom + o->paddingBottom();
 
     int buttonX;
     if (r.right() - r.x() < buttonWidth) {
@@ skipping 15 matching lines @@
     RECT renderRect = helper.rect();
     gfx::NativeTheme::instance()->PaintMenuList(hdc,
                                                 CP_DROPDOWNBUTTON,
                                                 determineState(o),
                                                 determineClassicState(o),
                                                 &renderRect);
     canvas->endPlatformPaint();
     return false;
 }
 
+void RenderThemeWin::adjustMenuListButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
+{
+    adjustMenuListStyle(selector, style, e);
+}
+
 // Used to paint styled menulists (i.e. with a non-default border)
 bool RenderThemeWin::paintMenuListButton(RenderObject* o, const RenderObject::PaintInfo& i, const IntRect& r)
 {
     return paintMenuList(o, i, r);
 }
 
 int RenderThemeWin::popupInternalPaddingLeft(RenderStyle* style) const
 {
     return menuListInternalPadding(style, LeftPadding);
 }
 
 int RenderThemeWin::popupInternalPaddingRight(RenderStyle* style) const
 {
     return menuListInternalPadding(style, RightPadding);
 }
 
 int RenderThemeWin::popupInternalPaddingTop(RenderStyle* style) const
 {
     return menuListInternalPadding(style, TopPadding);
 }
 
 int RenderThemeWin::popupInternalPaddingBottom(RenderStyle* style) const
 {
     return menuListInternalPadding(style, BottomPadding);
 }
 
-// Hacks for using Mac menu list metrics when in layout test mode.
-static int layoutTestMenuListInternalPadding(RenderStyle* style, int paddingType)
+void RenderThemeWin::adjustButtonInnerStyle(RenderStyle* style) const
 {
-    if (style->appearance() == MenulistPart) {
-        return kLayoutTestMenuListInternalPadding[controlSizeForFont(style)][paddingType];
-    }
-    if (style->appearance() == MenulistButtonPart) {
-        if (paddingType == RightPadding) {
-            const float baseArrowWidth = 5.0f;
-            float fontScale = style->fontSize() / kLayoutTestBaseFontSize;
-            float arrowWidth = ceilf(baseArrowWidth * fontScale);
-
-            int arrowPaddingLeft = 6;
-            int arrowPaddingRight = 6;
-            int paddingBeforeSeparator = 4;
-            // Add 2 for separator space, seems to match RenderThemeMac::paintMenuListButton.
-            return static_cast<int>(arrowWidth + arrowPaddingLeft + arrowPaddingRight +
-                                    paddingBeforeSeparator);
-        } else {
-            return kLayoutTestStyledMenuListInternalPadding[paddingType];
-        }
-    }
-    return 0;
-}
-
-int RenderThemeWin::menuListInternalPadding(RenderStyle* style, int paddingType) const
-{
-    if (ChromiumBridge::layoutTestMode()) {
-        return layoutTestMenuListInternalPadding(style, paddingType);
-    }
-
-    // This internal padding is in addition to the user-supplied padding.
-    // Matches the FF behavior.
-    int padding = kStyledMenuListInternalPadding[paddingType];
-
-    // Reserve the space for right arrow here. The rest of the padding is 
-    // set by adjustMenuListStyle, since PopMenuWin.cpp uses the padding from
-    // RenderMenuList to lay out the individual items in the popup.
-    // If the MenuList actually has appearance "NoAppearance", then that means
-    // we don't draw a button, so don't reserve space for it.
-    const int bar_type = style->direction() == LTR ? RightPadding : LeftPadding;
-    if (paddingType == bar_type && style->appearance() != NoPart)
-        padding += ScrollbarTheme::nativeTheme()->scrollbarThickness();
-
-    return padding;
-}
-
-void RenderThemeWin::adjustButtonInnerStyle(RenderStyle* style) const 
-{ 
     // This inner padding matches Firefox.
     style->setPaddingTop(Length(1, Fixed));
     style->setPaddingRight(Length(3, Fixed));
     style->setPaddingBottom(Length(1, Fixed));
     style->setPaddingLeft(Length(3, Fixed));
 }
 
-void RenderThemeWin::setButtonPadding(RenderStyle* style) const
-{  
-    if (ChromiumBridge::layoutTestMode()) {
-        setFixedPadding(style, kLayoutTestButtonPadding);
-
-    } else if (!style->width().isAuto()) {
-        // We need to set the minimum padding for the buttons, or else they
-        // render too small and clip the button face text. The right way to do
-        // this is to ask window's theme manager to give us the minimum 
-        // (TS_MIN) size for the part. As a failsafe we set at least 
-        // kDefaultButtonPadding because zero just looks bad.
-        Length minXPadding(kDefaultButtonPadding, Fixed);
-        Length minYPadding(kDefaultButtonPadding, Fixed);
-        // Find minimum padding.
-        getMinimalButtonPadding(&minXPadding);
-
-        // Set the minimum padding.
-        if (style->paddingLeft().value() < minXPadding.value()) {
-            style->setPaddingLeft(minXPadding);
-        }
-        if (style->paddingRight().value() < minXPadding.value()) {
-            style->setPaddingRight(minXPadding);
-        }
-        if (style->paddingBottom().value() < minYPadding.value()) { 
-            style->setPaddingBottom(minYPadding);
-        }
-        if (style->paddingTop().value() < minYPadding.value()) { 
-            style->setPaddingTop(minYPadding);
-        }
-    }
-}
-
 void RenderThemeWin::adjustSliderThumbSize(RenderObject* o) const
 {
     if (ChromiumBridge::layoutTestMode()) {
         if (o->style()->appearance() == SliderThumbHorizontalPart ||
             o->style()->appearance() == SliderThumbVerticalPart) {
             o->style()->setWidth(Length(kLayoutTestSliderThumbWidth, Fixed));
             o->style()->setHeight(Length(kLayoutTestSliderThumbHeight, Fixed));
         }
     }
 }
@@ skipping 20 matching lines @@
         // Override padding size to match AppKit text positioning.
         style->setPaddingLeft(Length(kLayoutTestSearchFieldPadding, Fixed));
         style->setPaddingRight(Length(kLayoutTestSearchFieldPadding, Fixed));
         style->setPaddingTop(Length(kLayoutTestSearchFieldPadding, Fixed));
         style->setPaddingBottom(Length(kLayoutTestSearchFieldPadding, Fixed));
 
         style->setBoxShadow(0);
     }
 }
 
-static const IntSize* layoutTestCancelButtonSizes()
-{
-    static const IntSize sizes[3] = { IntSize(16, 13), IntSize(13, 11), IntSize(13, 9) };
-    return sizes;
-}
-
-static const IntSize* layoutTestResultsButtonSizes()
-{
-    static const IntSize sizes[3] = { IntSize(19, 13), IntSize(17, 11), IntSize(17, 9) };
-    return sizes;
-}
-
 void RenderThemeWin::adjustSearchFieldCancelButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
 {
     if (ChromiumBridge::layoutTestMode()) {
         IntSize size = layoutTestCancelButtonSizes()[controlSizeForFont(style)];
         style->setWidth(Length(size.width(), Fixed));
         style->setHeight(Length(size.height(), Fixed));
         style->setBoxShadow(0);
     }
 }
 
@@ skipping 20 matching lines @@
 void RenderThemeWin::adjustSearchFieldResultsButtonStyle(CSSStyleSelector* selector, RenderStyle* style, Element* e) const
 {
     if (ChromiumBridge::layoutTestMode()) {
         IntSize size = layoutTestResultsButtonSizes()[controlSizeForFont(style)];
         style->setWidth(Length(size.width() + kLayoutTestResultsArrowWidth, Fixed));
         style->setHeight(Length(size.height(), Fixed));
         style->setBoxShadow(0);
     }
 }
 
+// static
+void RenderThemeWin::setDefaultFontSize(int fontSize) {
+    DefaultFontSize = static_cast<float>(fontSize);
+
+    // Reset cached fonts.
+    smallSystemFont = menuFont = labelFont = FontDescription();
+}
+
+unsigned RenderThemeWin::determineState(RenderObject* o)
+{
+    unsigned result = TS_NORMAL;
+    ControlPart appearance = o->style()->appearance();
+    if (!isEnabled(o))
+        result = TS_DISABLED;
+    else if (isReadOnlyControl(o) && (TextFieldPart == appearance || TextAreaPart == appearance))
+        result = ETS_READONLY; // Readonly is supported on textfields.
+    else if (isPressed(o)) // Active overrides hover and focused.
+        result = TS_PRESSED;
+    else if (supportsFocus(appearance) && isFocused(o))
+        result = ETS_FOCUSED;
+    else if (isHovered(o))
+        result = TS_HOT;
+    if (isChecked(o))
+        result += 4; // 4 unchecked states, 4 checked states.
+    return result;
+}
+
+unsigned RenderThemeWin::determineClassicState(RenderObject* o)
+{
+    unsigned result = 0;
+    if (!isEnabled(o))
+        result = DFCS_INACTIVE;
+    else if (isPressed(o)) // Active supersedes hover
+        result = DFCS_PUSHED;
+    else if (isHovered(o))
+        result = DFCS_HOT;
+    if (isChecked(o))
+        result |= DFCS_CHECKED;
+    return result;
+}
+
+ThemeData RenderThemeWin::getThemeData(RenderObject* o)
+{
+    ThemeData result;
+    switch (o->style()->appearance()) {
+        case PushButtonPart:
+        case ButtonPart:
+            result.m_part = BP_PUSHBUTTON;
+            result.m_classicState = DFCS_BUTTONPUSH;
+            break;
+        case CheckboxPart:
+            result.m_part = BP_CHECKBOX;
+            result.m_classicState = DFCS_BUTTONCHECK;
+            break;
+        case RadioPart:
+            result.m_part = BP_RADIOBUTTON;
+            result.m_classicState = DFCS_BUTTONRADIO;
+            break;
+        case ListboxPart:
+        case MenulistPart:
+        case TextFieldPart:
+        case TextAreaPart:
+            result.m_part = ETS_NORMAL;
+            break;
+    }
+
+    result.m_state = determineState(o);
+    result.m_classicState |= determineClassicState(o);
+
+    return result;
+}
+
+bool RenderThemeWin::paintTextFieldInternal(RenderObject* o,
+                                            const RenderObject::PaintInfo& i,
+                                            const IntRect& r,
+                                            bool drawEdges)
+{
+    // Nasty hack to make us not paint the border on text fields with a
+    // border-radius. Webkit paints elements with border-radius for us.
+    // TODO(ojan): Get rid of this if-check once we can properly clip rounded
+    // borders: http://b/1112604 and http://b/1108635
+    // TODO(ojan): make sure we do the right thing if css background-clip is
+    // set.
+    if (o->style()->hasBorderRadius())
+        return false;
+
+    const ThemeData& themeData = getThemeData(o);
+
+    WebCore::ThemeHelperWin helper(i.context, r);
+    gfx::PlatformCanvas* canvas = helper.context()->platformContext()->canvas();
+
+    HDC hdc = canvas->beginPlatformPaint();
+    COLORREF clr = gfx::SkColorToCOLORREF(o->style()->backgroundColor().rgb());
+    RECT renderRect = helper.rect();
+
+    gfx::NativeTheme::instance()->PaintTextField(hdc,
+                                                 themeData.m_part,
+                                                 themeData.m_state,
+                                                 themeData.m_classicState,
+                                                 &renderRect,
+                                                 clr,
+                                                 true,
+                                                 drawEdges);
+    canvas->endPlatformPaint();
+    return false;
+}
+
+void RenderThemeWin::setButtonPadding(RenderStyle* style) const
+{
+    if (ChromiumBridge::layoutTestMode()) {
+        setFixedPadding(style, kLayoutTestButtonPadding);
+
+    } else if (!style->width().isAuto()) {
+        // We need to set the minimum padding for the buttons, or else they
+        // render too small and clip the button face text. The right way to do
+        // this is to ask window's theme manager to give us the minimum
+        // (TS_MIN) size for the part. As a failsafe we set at least
+        // kDefaultButtonPadding because zero just looks bad.
+        Length minXPadding(kDefaultButtonPadding, Fixed);
+        Length minYPadding(kDefaultButtonPadding, Fixed);
+        // Find minimum padding.
+        getMinimalButtonPadding(&minXPadding);
+
+        // Set the minimum padding.
+        if (style->paddingLeft().value() < minXPadding.value()) {
+            style->setPaddingLeft(minXPadding);
+        }
+        if (style->paddingRight().value() < minXPadding.value()) {
+            style->setPaddingRight(minXPadding);
+        }
+        if (style->paddingBottom().value() < minYPadding.value()) {
+            style->setPaddingBottom(minYPadding);
+        }
+        if (style->paddingTop().value() < minYPadding.value()) {
+            style->setPaddingTop(minYPadding);
+        }
+    }
+}
+
 void RenderThemeWin::getMinimalButtonPadding(Length* minXPadding) const {
     // TODO(maruel): This get messy if 1. the theme change; 2. we are serializing.
     SIZE size;
     if (SUCCEEDED(gfx::NativeTheme::instance()->GetThemePartSize(
             gfx::NativeTheme::BUTTON, 0, BP_PUSHBUTTON, PBS_NORMAL, 0, TS_MIN,
             &size))) {
         *minXPadding = Length(size.cx, Fixed);
     }
 }
 
-// static
-void RenderThemeWin::setDefaultFontSize(int fontSize) {
-    DefaultFontSize = static_cast<float>(fontSize);
+int RenderThemeWin::menuListInternalPadding(RenderStyle* style, int paddingType) const
+{
+    if (ChromiumBridge::layoutTestMode()) {
+        return layoutTestMenuListInternalPadding(style, paddingType);
+    }
 
-    // Reset cached fonts.
-    smallSystemFont = menuFont = labelFont = FontDescription();
+    // This internal padding is in addition to the user-supplied padding.
+    // Matches the FF behavior.
+    int padding = kStyledMenuListInternalPadding[paddingType];
+
+    // Reserve the space for right arrow here. The rest of the padding is
+    // set by adjustMenuListStyle, since PopMenuWin.cpp uses the padding from
+    // RenderMenuList to lay out the individual items in the popup.
+    // If the MenuList actually has appearance "NoAppearance", then that means
+    // we don't draw a button, so don't reserve space for it.
+    const int bar_type = style->direction() == LTR ? RightPadding : LeftPadding;
+    if (paddingType == bar_type && style->appearance() != NoPart)
+        padding += ScrollbarTheme::nativeTheme()->scrollbarThickness();
+
+    return padding;
 }
 
-}
+}  // namespace WebCore