Small change to support the implementation of #7358

svn changeset:20940/svn branch:6.7

src/com/vaadin/ui/Table.java

@@ -1617,11 +1617,7 @@ public class Table extends AbstractSelect implements Action.Container,
                 }
 
                 if (value instanceof Component) {
-                    if (oldVisibleComponents == null
-                            || !oldVisibleComponents.contains(value)) {
-                        ((Component) value).setParent(this);
-                    }
-                    visibleComponents.add((Component) value);
+                    registerComponent((Component) value);
                 }
                 cells[CELL_FIRSTCOL + j][i] = value;
             }
@@ -1658,6 +1654,13 @@ public class Table extends AbstractSelect implements Action.Container,
         return cells;
     }
 
+    protected void registerComponent(Component component) {
+        if (component.getParent() != this) {
+            component.setParent(this);
+        }
+        visibleComponents.add(component);
+    }
+
     private void listenProperty(Property p,
             HashSet<Property> oldListenedProperties) {
         if (p instanceof Property.ValueChangeNotifier) {