Move deprecated getComponentIterator() from HasComponents to implementing classes (#10085)

Change-Id: Iedcb7b51f6d2a095b402a7141dc906b112ee6fe7

server/src/com/vaadin/ui/ComponentContainer.java

@@ -17,6 +17,7 @@
 package com.vaadin.ui;
 
 import java.io.Serializable;
+import java.util.Iterator;
 
 /**
  * Extension to the {@link Component} interface which adds to it the capacity to
@@ -69,6 +70,18 @@ public interface ComponentContainer extends HasComponents {
      */
     public void replaceComponent(Component oldComponent, Component newComponent);
 
+    /**
+     * Gets an iterator to the collection of contained components. Using this
+     * iterator it is possible to step through all components contained in this
+     * container.
+     * 
+     * @return the component iterator.
+     * 
+     * @deprecated As of 7.0, use {@link #iterator()} instead.
+     */
+    @Deprecated
+    public Iterator<Component> getComponentIterator();
+
     /**
      * Gets the number of children this {@link ComponentContainer} has. This
      * must be symmetric with what {@link #getComponentIterator()} returns.

server/src/com/vaadin/ui/CustomField.java

@@ -146,14 +146,17 @@ public abstract class CustomField<T> extends AbstractField<T> implements
         }
     }
 
-    @Override
+    /**
+     * @deprecated As of 7.0, use {@link #iterator()} instead.
+     */
+    @Deprecated
     public Iterator<Component> getComponentIterator() {
-        return new ComponentIterator();
+        return iterator();
     }
 
     @Override
     public Iterator<Component> iterator() {
-        return getComponentIterator();
+        return new ComponentIterator();
     }
 
     @Override

server/src/com/vaadin/ui/Form.java

@@ -1306,7 +1306,7 @@ public class Form extends AbstractField<Object> implements Item.Editor,
 
     @Override
     public Iterator<Component> iterator() {
-        return getComponentIterator();
+        return new ComponentIterator();
     }
 
     /**
@@ -1355,9 +1355,12 @@ public class Form extends AbstractField<Object> implements Item.Editor,
         }
     }
 
-    @Override
+    /**
+     * @deprecated As of 7.0, use {@link #iterator()} instead.
+     */
+    @Deprecated
     public Iterator<Component> getComponentIterator() {
-        return new ComponentIterator();
+        return iterator();
     }
 
     public int getComponentCount() {

server/src/com/vaadin/ui/HasComponents.java

@@ -32,11 +32,9 @@ public interface HasComponents extends Component, Iterable<Component> {
      * container.
      * 
      * @return the component iterator.
-     * 
-     * @deprecated Use {@link #iterator()} instead.
      */
-    @Deprecated
-    public Iterator<Component> getComponentIterator();
+    @Override
+    public Iterator<Component> iterator();
 
     /**
      * Checks if the child component is visible. This method allows hiding a

server/src/com/vaadin/ui/Table.java

@@ -5564,19 +5564,21 @@ public class Table extends AbstractSelect implements Action.Container,
 
     @Override
     public Iterator<Component> iterator() {
-        return getComponentIterator();
-    }
-
-    @Override
-    public Iterator<Component> getComponentIterator() {
         if (visibleComponents == null) {
             Collection<Component> empty = Collections.emptyList();
             return empty.iterator();
         }
-
         return visibleComponents.iterator();
     }
 
+    /**
+     * @deprecated As of 7.0, use {@link #iterator()} instead.
+     */
+    @Deprecated
+    public Iterator<Component> getComponentIterator() {
+        return iterator();
+    }
+
     @Override
     public boolean isComponentVisible(Component childComponent) {
         return true;