From: Leif Åstrand <leif@vaadin.com>
Date: Tue, 7 Feb 2012 13:07:09 +0000 (+0200)
Subject: Make container instead of child depend on child caption (#8313)
X-Git-Tag: 7.0.0.alpha2~434^2~83
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d434bfd1d35f576a3b9ae5f3288464990a6106dd;p=vaadin-framework.git

Make container instead of child depend on child caption (#8313)
---

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VMeasuringOrderedLayoutPaintable.java b/src/com/vaadin/terminal/gwt/client/ui/VMeasuringOrderedLayoutPaintable.java
index f1838cf30e..c13ebc87f3 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VMeasuringOrderedLayoutPaintable.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VMeasuringOrderedLayoutPaintable.java
@@ -31,10 +31,7 @@ public abstract class VMeasuringOrderedLayoutPaintable extends
                 getWidgetForPaintable().addCaption(caption, widget);
                 getWidgetForPaintable().captions.put(component, caption);
 
-                MeasureManager.MeasuredSize measuredSize = component
-                        .getMeasuredSize();
-
-                measuredSize.registerDependency(caption.getElement());
+                getMeasuredSize().registerDependency(caption.getElement());
             }
             caption.updateCaption(uidl);
         } else {
@@ -42,9 +39,8 @@ public abstract class VMeasuringOrderedLayoutPaintable extends
                     .remove(component);
             if (removedCaption != null) {
                 getWidgetForPaintable().remove(removedCaption);
-                MeasureManager.MeasuredSize measuredSize = component
-                        .getMeasuredSize();
-                measuredSize.deRegisterDependency(removedCaption.getElement());
+                getMeasuredSize().deRegisterDependency(
+                        removedCaption.getElement());
             }
         }
     }
@@ -121,18 +117,15 @@ public abstract class VMeasuringOrderedLayoutPaintable extends
         if (caption == null) {
             return 0;
         } else {
-            MeasureManager.MeasuredSize measuredSize = child.getMeasuredSize();
-            return measuredSize.getDependencyWidth(caption.getElement());
+            return getMeasuredSize().getDependencyWidth(caption.getElement());
         }
     }
 
     private int getCaptionHeight(VPaintableWidget child) {
-
         VCaption caption = getWidgetForPaintable().captions.get(child);
         if (caption != null) {
-            MeasureManager.MeasuredSize measuredSize = child.getMeasuredSize();
-            int captionHeight = measuredSize.getDependencyHeight(caption
-                    .getElement());
+            int captionHeight = getMeasuredSize().getDependencyHeight(
+                    caption.getElement());
 
             caption.getElement().getStyle()
                     .setMarginTop(-captionHeight, Unit.PX);