From: Artur Signell <artur@vaadin.com>
Date: Tue, 20 Mar 2012 12:44:06 +0000 (+0200)
Subject: Code cleanup
X-Git-Tag: 7.0.0.alpha2~268
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=49c33ff2d04c04dd21c9244d673b934ac340eacb;p=vaadin-framework.git

Code cleanup
---

diff --git a/src/com/vaadin/ui/AbstractComponent.java b/src/com/vaadin/ui/AbstractComponent.java
index 4d12e0ed27..ad91f342ba 100644
--- 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();
         }
     }