Remove unnecessary if check from RenderListBox::paintItemForeground.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Nov 2011 15:04:58 +0000 (15:04 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Nov 2011 15:04:58 +0000 (15:04 +0000)
https://bugs.webkit.org/show_bug.cgi?id=72488

Patch by Antaryami Pandia <antaryami.pandia@motorola.com> on 2011-11-16
Reviewed by Andreas Kling.

* rendering/RenderListBox.cpp:
(WebCore::RenderListBox::paintItemForeground):

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderListBox.cpp

index 970eff14d522ab4167e4ea9714badf36c04e5145..dbd0c8a3f7fb43fb1b0bc99530690e38a89f4328 100644 (file)
@@ -1,3 +1,13 @@
+2011-11-16  Antaryami Pandia  <antaryami.pandia@motorola.com>
+
+        Remove unnecessary if check from RenderListBox::paintItemForeground.
+        https://bugs.webkit.org/show_bug.cgi?id=72488
+
+        Reviewed by Andreas Kling.
+
+        * rendering/RenderListBox.cpp:
+        (WebCore::RenderListBox::paintItemForeground):
+
 2011-11-15  Simon Hausmann  <simon.hausmann@nokia.com>
 
         [Qt] Centralize hide_symbols and ensure all libs are built with symbol visibility & bsymbolic_functions
index 0b758a49021b95909958f6019d6afa037f6ad5a6..5e52d15b7e8ed168ef1d90cd1901d56a89e1faa9 100644 (file)
@@ -420,8 +420,7 @@ void RenderListBox::paintItemForeground(PaintInfo& paintInfo, const LayoutPoint&
     }
 
     // Draw the item text
-    if (itemStyle->visibility() != HIDDEN)
-        paintInfo.context->drawBidiText(itemFont, textRun, r.location());
+    paintInfo.context->drawBidiText(itemFont, textRun, r.location());
 }
 
 void RenderListBox::paintItemBackground(PaintInfo& paintInfo, const LayoutPoint& paintOffset, int listIndex)