From a328f692ffda3c6868fd1e889240f74aac607e6f Mon Sep 17 00:00:00 2001
From: Artur Signell <artur@vaadin.com>
Date: Tue, 7 Feb 2012 18:13:58 +0200
Subject: [PATCH] Avoid caching parent for now as the cache is never cleared

---
 .../terminal/gwt/client/ui/VAbstractPaintableWidget.java  | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

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;
             }
         }
 
-- 
2.39.5