Code cleanup

diff --git a/src/com/vaadin/ui/AbstractComponent.java b/src/com/vaadin/ui/AbstractComponent.java
index 4d12e0ed27564cf96adfc63b0854f5649a9e47f0..ad91f342bada127210a47f0152d03ca58e0dec25 100644 (file)
--- a/src/com/vaadin/ui/AbstractComponent.java
+++ b/src/com/vaadin/ui/AbstractComponent.java
@@ -438,14 +438,16 @@ public abstract class AbstractComponent implements Component, MethodEventSource
      * @see com.vaadin.ui.Component#setVisible(boolean)
      */
     public void setVisible(boolean visible) {
-        if (getState().isVisible() != visible) {
-            getState().setVisible(visible);
-            requestRepaint();
-            if (getParent() != null) {
-                // Must always repaint the parent (at least the hierarchy) when
-                // visibility of a child component changes.
-                getParent().requestRepaint();
-            }
+        if (getState().isVisible() == visible) {
+            return;
+        }
+
+        getState().setVisible(visible);
+        requestRepaint();
+        if (getParent() != null) {
+            // Must always repaint the parent (at least the hierarchy) when
+            // visibility of a child component changes.
+            getParent().requestRepaint();
         }
     }