Merge changes from origin/7.1

30e9b7c Add a test for @mixin { @font-face {...} } (#11649)
0a2cf2c Prevent showing "Session Expired" notification if navigating away (#12298)
8efc385 Fixes NPE in TreeTable.setContainerDS(null) (#12928)
23815e6 Remove broken JSch check
c308d8a Fixes double scrollbar issue in browserframes (#11780) Fixed by adding font-size 0 to the base theme.

Change-Id: I64ae167e8e6b61205718998b80d534fb2cf0cb3d

2 files changed, 55 insertions, 0 deletions

diff --git a/uitest/src/com/vaadin/tests/components/browserframe/BrowserFrameDoubleScrollbars.java b/uitest/src/com/vaadin/tests/components/browserframe/BrowserFrameDoubleScrollbars.java
new file mode 100644
index 0000000000..5151ad5dd3
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/components/browserframe/BrowserFrameDoubleScrollbars.java
@@ -0,0 +1,40 @@
+package com.vaadin.tests.components.browserframe;
+
+import com.vaadin.server.ExternalResource;
+import com.vaadin.server.VaadinRequest;
+import com.vaadin.tests.components.AbstractTestUI;
+import com.vaadin.ui.BrowserFrame;
+import com.vaadin.ui.TabSheet;
+
+public class BrowserFrameDoubleScrollbars extends AbstractTestUI {
+
+    @Override
+    protected Integer getTicketNumber() {
+        return 11780;
+    }
+
+    @Override
+    protected void setup(VaadinRequest request) {
+
+        getLayout().setHeight("100%");
+        getLayout().setSizeFull();
+        getLayout().getParent().setSizeFull();
+
+        TabSheet tabs = new TabSheet();
+        tabs.setSizeFull();
+        getLayout().addComponent(tabs);
+
+        BrowserFrame help = new BrowserFrame();
+        help.setSizeFull();
+        help.setSource(new ExternalResource("/statictestfiles/long-html.htm"));
+
+        tabs.addComponent(help);
+
+    }
+
+    @Override
+    protected String getTestDescription() {
+        return "Embedded browser causes second scrollbar";
+    }
+
+}
diff --git a/uitest/src/com/vaadin/tests/components/browserframe/BrowserFrameDoubleScrollbarsTest.java b/uitest/src/com/vaadin/tests/components/browserframe/BrowserFrameDoubleScrollbarsTest.java
new file mode 100644
index 0000000000..ade43735a7
--- /dev/null
+++ b/uitest/src/com/vaadin/tests/components/browserframe/BrowserFrameDoubleScrollbarsTest.java
@@ -0,0 +1,15 @@
+package com.vaadin.tests.components.browserframe;
+
+import org.junit.Test;
+
+import com.vaadin.tests.tb3.MultiBrowserTest;
+
+public class BrowserFrameDoubleScrollbarsTest extends MultiBrowserTest {
+
+    @Test
+    public void testWindowRepositioning() throws Exception {
+        openTestURL();
+
+        compareScreen("BrowserFrameDoubleScrollbars");
+    }
+}