2011-02-04 Xiyuan Xia <xiyuan@chromium.org>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 5 Feb 2011 02:19:43 +0000 (02:19 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 5 Feb 2011 02:19:43 +0000 (02:19 +0000)
        Reviewed by Tony Chang.

        [Chromium] Option text in select popup does not align with menulist button text
        https://bugs.webkit.org/show_bug.cgi?id=53632

        This makes clientPaddingLeft and  clientPaddingRight return
        the additional m_innerBlock's padding so that the popup item text
        aligns with the menulist button text.

        * rendering/RenderMenuList.cpp:
        (WebCore::RenderMenuList::clientPaddingLeft):
        (WebCore::RenderMenuList::clientPaddingRight):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@77716 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderMenuList.cpp

index 7aa4341c1b82148d835785a0fc959299c824aa4e..4ab54d012f1c560e2bf4709ef19466b738039a9b 100644 (file)
@@ -1,3 +1,18 @@
+2011-02-04  Xiyuan Xia  <xiyuan@chromium.org>
+
+        Reviewed by Tony Chang.
+
+        [Chromium] Option text in select popup does not align with menulist button text
+        https://bugs.webkit.org/show_bug.cgi?id=53632
+
+        This makes clientPaddingLeft and  clientPaddingRight return
+        the additional m_innerBlock's padding so that the popup item text
+        aligns with the menulist button text.
+
+        * rendering/RenderMenuList.cpp:
+        (WebCore::RenderMenuList::clientPaddingLeft):
+        (WebCore::RenderMenuList::clientPaddingRight):
+
 2011-02-04  Anders Carlsson  <andersca@apple.com>
 
         Reviewed by Sam Weinig and Beth Dakin.
index c45e60aa55d73ce7976b5b59e2a9cea33c043ae8..0e194b7c74dce35898b5d02747ee1ac6216c8549 100644 (file)
@@ -486,7 +486,7 @@ int RenderMenuList::clientInsetRight() const
 
 int RenderMenuList::clientPaddingLeft() const
 {
-    return paddingLeft();
+    return paddingLeft() + m_innerBlock->paddingLeft();
 }
 
 const int endOfLinePadding = 2;
@@ -502,7 +502,7 @@ int RenderMenuList::clientPaddingRight() const
 
     // If the appearance isn't MenulistPart, then the select is styled (non-native), so
     // we want to return the user specified padding.
-    return paddingRight();
+    return paddingRight() + m_innerBlock->paddingRight();
 }
 
 int RenderMenuList::listSize() const