From: Artur Signell <artur@vaadin.com>
Date: Tue, 7 Feb 2012 16:13:58 +0000 (+0200)
Subject: Avoid caching parent for now as the cache is never cleared
X-Git-Tag: 7.0.0.alpha2~455
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=a328f692ffda3c6868fd1e889240f74aac607e6f;p=vaadin-framework.git

Avoid caching parent for now as the cache is never cleared
---

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VAbstractPaintableWidget.java b/src/com/vaadin/terminal/gwt/client/ui/VAbstractPaintableWidget.java
index 6813c225d3..efe6896e9b 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VAbstractPaintableWidget.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VAbstractPaintableWidget.java
@@ -22,7 +22,6 @@ public abstract class VAbstractPaintableWidget implements VPaintableWidget {
     private Widget widget;
     private ApplicationConnection connection;
     private String id;
-    private VPaintableWidgetContainer parent;
 
     /* State variables */
     private boolean enabled = true;
@@ -91,10 +90,6 @@ public abstract class VAbstractPaintableWidget implements VPaintableWidget {
     }
 
     public VPaintableWidgetContainer getParent() {
-        if (parent != null) {
-            return parent;
-        }
-
         // FIXME: Hierarchy should be set by framework instead of looked up here
         VPaintableMap paintableMap = VPaintableMap.get(getConnection());
 
@@ -102,9 +97,8 @@ public abstract class VAbstractPaintableWidget implements VPaintableWidget {
         while (w != null) {
             w = w.getParent();
             if (paintableMap.isPaintable(w)) {
-                parent = (VPaintableWidgetContainer) paintableMap
+                return (VPaintableWidgetContainer) paintableMap
                         .getPaintable(w);
-                return parent;
             }
         }