--- /dev/null
+package com.vaadin.tests.components.orderedlayout;
+
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.ui.Button;
+import com.vaadin.ui.ComponentContainer;
+import com.vaadin.ui.CssLayout;
+import com.vaadin.ui.Label;
+import com.vaadin.ui.Panel;
+import com.vaadin.ui.TextField;
+import com.vaadin.ui.UI;
+import com.vaadin.ui.VerticalLayout;
+
+/**
+ * HorizontalLayout and VerticalLayout should not leak caption elements via
+ * listeners when removing components from a layout.
+ *
+ * @since 7.1.13
+ * @author Vaadin Ltd
+ */
+public class CaptionLeak extends UI {
+
+ public static final String USAGE = "Open this UI with ?debug and count"
+ + " measured non-connector elements after setting leaky and non leaky"
+ + " content.";
+
+ @Override
+ public void init(VaadinRequest req) {
+ final VerticalLayout root = new VerticalLayout();
+ setContent(root);
+ Label usage = new Label(USAGE);
+ Panel parent = new Panel();
+ Button setLeakyContent = makeButton("Set leaky content", parent,
+ VerticalLayout.class);
+ Button setNonLeakyContent = makeButton("Set non leaky content", parent,
+ CssLayout.class);
+ root.addComponents(usage, setLeakyContent, setNonLeakyContent, parent);
+ }
+
+ private Button makeButton(String caption, final Panel parent,
+ final Class<? extends ComponentContainer> targetClass) {
+ Button btn = new Button(caption);
+ btn.setId(caption);
+ btn.addClickListener(new Button.ClickListener() {
+ @Override
+ public void buttonClick(Button.ClickEvent event) {
+ try {
+ ComponentContainer target = targetClass.newInstance();
+ for (int i = 0; i < 61; i++) {
+ target.addComponent(new TextField("Test"));
+ }
+ parent.setContent(target);
+ } catch (InstantiationException e) {
+ throw new RuntimeException(e);
+ } catch (IllegalAccessException e) {
+ throw new RuntimeException(e);
+ }
+ }
+ });
+ return btn;
+ }
+
+}
--- /dev/null
+/*
+ * Copyright 2000-2013 Vaadin Ltd.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package com.vaadin.tests.components.orderedlayout;
+
+import org.junit.Test;
+import org.openqa.selenium.By;
+
+import com.vaadin.tests.tb3.MultiBrowserTest;
+
+public class CaptionLeakTest extends MultiBrowserTest {
+
+ @Test
+ public void testCaptionLeak() throws Exception {
+ setDebug(true);
+ openTestURL();
+
+ // this should be present
+ // 3 general non-connector elements, none accumulated on click
+ getDriver()
+ .findElement(
+ By.xpath("//span[text() = 'Measured 3 non connector elements']"));
+
+ getDriver().findElement(By.xpath("//button[@title = 'Clear log']"))
+ .click();
+ getDriver().findElement(By.id("Set leaky content")).click();
+
+ getDriver()
+ .findElement(
+ By.xpath("//span[text() = 'Measured 3 non connector elements']"));
+
+ // nothing accumulates over clicks
+ getDriver().findElement(By.xpath("//button[@title = 'Clear log']"))
+ .click();
+ getDriver().findElement(By.id("Set leaky content")).click();
+ getDriver()
+ .findElement(
+ By.xpath("//span[text() = 'Measured 3 non connector elements']"));
+ }
+
+ @Test
+ public void testNoCaptionLeak() throws Exception {
+ setDebug(true);
+ openTestURL();
+
+ getDriver().findElement(By.xpath("//button[@title = 'Clear log']"))
+ .click();
+ getDriver().findElement(By.id("Set non leaky content")).click();
+
+ // this should be present
+ // 3 general non-connector elements, none accumulated on click
+ getDriver()
+ .findElement(
+ By.xpath("//span[text() = 'Measured 3 non connector elements']"));
+ }
+}