From 88af61e7576af2220cf857692a484c7e1f2a6326 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Leif=20=C3=85strand?= <leif@vaadin.com>
Date: Fri, 14 Sep 2012 14:52:53 +0300
Subject: [PATCH] Add comment explaining why beforeClientResponse is used

---
 server/src/com/vaadin/ui/CssLayout.java | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/server/src/com/vaadin/ui/CssLayout.java b/server/src/com/vaadin/ui/CssLayout.java
index 401eafe6e9..172258a5fe 100644
--- a/server/src/com/vaadin/ui/CssLayout.java
+++ b/server/src/com/vaadin/ui/CssLayout.java
@@ -228,6 +228,11 @@ public class CssLayout extends AbstractLayout implements LayoutClickNotifier {
     @Override
     public void beforeClientResponse(boolean initial) {
         super.beforeClientResponse(initial);
+
+        // This is an obsolete hack that was required before Map<Conenctor, ?>
+        // was supported. The workaround is to instead use a Map<String, ?> with
+        // the connector id as the key, but that can only be used once the
+        // connector has been attached.
         getState().childCss.clear();
         for (Iterator<Component> ci = getComponentIterator(); ci.hasNext();) {
             Component child = ci.next();
-- 
2.39.5