1 files changed, 54 insertions, 0 deletions
diff --git a/uitest/src/com/vaadin/tests/components/orderedlayout/SpacingLeak.java b/uitest/src/com/vaadin/tests/components/orderedlayout/SpacingLeak.java
new file mode 100644
index 0000000000..647c187568
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/components/orderedlayout/SpacingLeak.java
@@ -0,0 +1,54 @@
+package com.vaadin.tests.components.orderedlayout;
+
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.ui.Button;
+import com.vaadin.ui.HorizontalLayout;
+import com.vaadin.ui.UI;
+import com.vaadin.ui.VerticalLayout;
+
+/**
+ * HorizontalLayout and VerticalLayout should not leak .v-spacing elements via
+ * listeners when removing components from a layout.
+ * 
+ * @since 7.1.12
+ * @author Vaadin Ltd
+ */
+public class SpacingLeak extends UI {
+
+    private HorizontalLayout spacingLayout;
+
+    @Override
+    public void init(VaadinRequest req) {
+        final VerticalLayout root = new VerticalLayout();
+        setContent(root);
+        root.setSizeUndefined();
+
+        final Button spacingButton = new Button("Add layout with spacing");
+        spacingButton.setId("addbutton");
+        root.addComponent(spacingButton);
+        spacingButton.addClickListener(new Button.ClickListener() {
+            @Override
+            public void buttonClick(Button.ClickEvent event) {
+                spacingLayout = new HorizontalLayout();
+                spacingLayout.setSpacing(true);
+                spacingLayout.setWidth("100%");
+
+                for (int i = 0; i < 100; ++i) {
+                    spacingLayout.addComponent(new Button("" + i));
+                }
+
+                root.addComponent(spacingLayout);
+            }
+        });
+
+        final Button removeButton = new Button("Remove layouts");
+        removeButton.setId("removebutton");
+        root.addComponent(removeButton);
+        removeButton.addClickListener(new Button.ClickListener() {
+            @Override
+            public void buttonClick(Button.ClickEvent event) {
+                root.removeComponent(spacingLayout);
+            }
+        });
+    }
+}