]> source.dussan.org Git - vaadin-framework.git/commitdiff
Removed unnecessary calls to unregisterPaintable
authorArtur Signell <artur@vaadin.com>
Wed, 4 Apr 2012 21:30:17 +0000 (00:30 +0300)
committerArtur Signell <artur@vaadin.com>
Wed, 4 Apr 2012 21:30:17 +0000 (00:30 +0300)
src/com/vaadin/terminal/gwt/client/ui/FormConnector.java
src/com/vaadin/terminal/gwt/client/ui/RootConnector.java
src/com/vaadin/terminal/gwt/client/ui/VPopupView.java
src/com/vaadin/terminal/gwt/client/ui/WindowConnector.java

index 2cc9c9968e8893b99dfb0434612c71679c41b0d6..4822ec734d41f82e23477f341f09212bcad96617 100644 (file)
@@ -10,7 +10,6 @@ import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.terminal.gwt.client.AbstractFieldState;
 import com.vaadin.terminal.gwt.client.ApplicationConnection;
 import com.vaadin.terminal.gwt.client.ComponentConnector;
-import com.vaadin.terminal.gwt.client.ConnectorMap;
 import com.vaadin.terminal.gwt.client.LayoutManager;
 import com.vaadin.terminal.gwt.client.Paintable;
 import com.vaadin.terminal.gwt.client.UIDL;
@@ -99,8 +98,6 @@ public class FormConnector extends AbstractComponentContainerConnector
                 getWidget().footer = newFooterWidget;
             } else if (newFooter != getWidget().footer) {
                 getWidget().remove(getWidget().footer);
-                client.unregisterPaintable(ConnectorMap.get(getConnection())
-                        .getConnector(getWidget().footer));
                 getWidget().add(newFooter.getWidget(),
                         getWidget().footerContainer);
             }
@@ -108,8 +105,6 @@ public class FormConnector extends AbstractComponentContainerConnector
         } else {
             if (getWidget().footer != null) {
                 getWidget().remove(getWidget().footer);
-                client.unregisterPaintable(ConnectorMap.get(getConnection())
-                        .getConnector(getWidget().footer));
             }
         }
 
@@ -122,8 +117,6 @@ public class FormConnector extends AbstractComponentContainerConnector
             getWidget().add(newLayoutWidget, getWidget().fieldContainer);
         } else if (getWidget().lo != newLayoutWidget) {
             // Layout has changed
-            client.unregisterPaintable(ConnectorMap.get(getConnection())
-                    .getConnector(getWidget().lo));
             getWidget().remove(getWidget().lo);
             getWidget().lo = newLayoutWidget;
             getWidget().add(newLayoutWidget, getWidget().fieldContainer);
index 8af9b39825c2623b4116a314ad9c8d07607ee122..392230aa7f55e74ae64dbe29602f84ac3851f5a7 100644 (file)
@@ -166,7 +166,6 @@ public class RootConnector extends AbstractComponentContainerConnector
         if (getWidget().layout != null) {
             if (layoutChanged) {
                 // remove old
-                client.unregisterPaintable(getWidget().layout);
                 if (childStateChangeHandlerRegistration != null) {
                     childStateChangeHandlerRegistration.removeHandler();
                     childStateChangeHandlerRegistration = null;
@@ -227,8 +226,6 @@ public class RootConnector extends AbstractComponentContainerConnector
         for (final Iterator<VWindow> rem = removedSubWindows.iterator(); rem
                 .hasNext();) {
             final VWindow w = rem.next();
-            client.unregisterPaintable(ConnectorMap.get(getConnection())
-                    .getConnector(w));
             getWidget().subWindows.remove(w);
             w.hide();
         }
index e6ca747b7c0b3982f4dd9bed73ee2780682f528c..c13c4339a5ee12ca4d851423a599e9ac3cb9d5f4 100644 (file)
@@ -228,7 +228,6 @@ public class VPopupView extends HTML {
         public void hide(boolean autoClosed) {
             hiding = true;
             syncChildren();
-            unregisterPaintables();
             if (popupComponentWidget != null && popupComponentWidget != loading) {
                 remove(popupComponentWidget);
             }
@@ -301,12 +300,6 @@ public class VPopupView extends HTML {
 
         }
 
-        public void unregisterPaintables() {
-            if (popupComponentPaintable != null) {
-                client.unregisterPaintable(popupComponentPaintable);
-            }
-        }
-
         public void setHideOnMouseOut(boolean hideOnMouseOut) {
             this.hideOnMouseOut = hideOnMouseOut;
         }
index 58ae583fa6ee399f2428eff7a8667a4529df2c4b..7c50b0f34f9529ce33fe121bc277270eb6d95427 100644 (file)
@@ -194,7 +194,6 @@ public class WindowConnector extends AbstractComponentContainerConnector
         if (getWidget().layout != null) {
             if (getWidget().layout != lo) {
                 // remove old
-                client.unregisterPaintable(getWidget().layout);
                 getWidget().contentPanel.remove(getWidget().layout.getWidget());
                 // add new
                 if (!showingUrl) {