From 97cb5b1cd160cfa9dc00e41b5206c83ce26a4c35 Mon Sep 17 00:00:00 2001
From: Artur Signell <artur@vaadin.com>
Date: Fri, 16 Mar 2012 20:14:51 +0200
Subject: [PATCH] Javadoc - mentioned that components must notify their Root
 when they are attached/detached

---
 src/com/vaadin/ui/Component.java | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/com/vaadin/ui/Component.java b/src/com/vaadin/ui/Component.java
index ae6fcd9378..c0c43fcec3 100644
--- a/src/com/vaadin/ui/Component.java
+++ b/src/com/vaadin/ui/Component.java
@@ -329,7 +329,6 @@ public interface Component extends ClientConnector, Paintable, VariableOwner,
      * is attached to the application, {@link #detach()} is called for the
      * component.
      * </p>
-     * 
      * <p>
      * This method is rarely called directly. The
      * {@link ComponentContainer#addComponent(Component)} method is normally
@@ -620,6 +619,11 @@ public interface Component extends ClientConnector, Paintable, VariableOwner,
      * application, the {@code attach()} is called immediately from
      * {@link #setParent(Component)}.
      * </p>
+     * <p>
+     * This method must call {@link Root#componentAttached(Component)} to let
+     * the Root know that a new Component has been attached.
+     * </p>
+     * 
      * 
      * <pre>
      * public class AttachExample extends CustomComponent {
@@ -655,6 +659,11 @@ public interface Component extends ClientConnector, Paintable, VariableOwner,
      * </p>
      * 
      * <p>
+     * This method must call {@link Root#componentDetached(Component)} to let
+     * the Root know that a new Component has been attached.
+     * </p>
+     * *
+     * <p>
      * The caller of this method is {@link #setParent(Component)} if the parent
      * is in the application. When the parent is detached from the application
      * it is its response to call {@link #detach()} for all the children and to
-- 
2.39.5