Mercurial > hg > release > icedtea-web-1.4
changeset 740:b5b5f59833e2
Reproducer for PR854: Resizing an applet several times causes 100% CPU load
author | Adam Domurad <adomurad@redhat.com> |
---|---|
date | Tue, 21 May 2013 09:31:57 -0400 |
parents | d340d2f2ac7c |
children | e281ea56466e |
files | ChangeLog tests/reproducers/simple/ResizeApplet/resources/ResizeApplet.html tests/reproducers/simple/ResizeApplet/srcs/ResizeApplet.java tests/reproducers/simple/ResizeApplet/testcases/ResizeAppletTests.java |
diffstat | 4 files changed, 204 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/ChangeLog Tue May 21 09:11:41 2013 -0400 +++ b/ChangeLog Tue May 21 09:31:57 2013 -0400 @@ -1,3 +1,13 @@ +2013-05-21 Adam Domurad <adomurad@redhat.com> + + Reproducer for PR854. + * tests/reproducers/simple/ResizeApplet/resources/ResizeApplet.html: + Resizes applet from Javascript. + * tests/reproducers/simple/ResizeApplet/srcs/ResizeApplet.java: + Simple applet with a few helper methods. + * tests/reproducers/simple/ResizeApplet/testcases/ResizeAppletTests.java: + Test applet resizing. + 2013-05-14 Adam Domurad <adomurad@redhat.com> Fix PR854: Resizing an applet several times causes 100% CPU load
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/tests/reproducers/simple/ResizeApplet/resources/ResizeApplet.html Tue May 21 09:31:57 2013 -0400 @@ -0,0 +1,57 @@ +<!-- + +This file is part of IcedTea. + +IcedTea is free software; you can redistribute it and/or modify +it under the terms of the GNU General Public License as published by +the Free Software Foundation; either version 2, or (at your option) +any later version. + +IcedTea is distributed in the hope that it will be useful, but +WITHOUT ANY WARRANTY; without even the implied warranty of +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +General Public License for more details. + +You should have received a copy of the GNU General Public License +along with IcedTea; see the file COPYING. If not, write to the +Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA +02110-1301 USA. + +Linking this library statically or dynamically with other modules is +making a combined work based on this library. Thus, the terms and +conditions of the GNU General Public License cover the whole +combination. + +As a special exception, the copyright holders of this library give you +permission to link this library with independent modules to produce an +executable, regardless of the license terms of these independent +modules, and to copy and distribute the resulting executable under +terms of your choice, provided that you also meet, for each linked +independent module, the terms and conditions of the license of that +module. An independent module is a module which is not derived from +or based on this library. If you modify this library, you may extend +this exception to your version of the library, but you are not +obligated to do so. If you do not wish to do so, delete this +exception statement from your version. + + --> +<html><head></head><body bgcolor="blue"> +<p><applet id="AppletId" code="ResizeApplet.class" archive="ResizeApplet.jar" codebase="." width="100" height="100"> +</applet></p> + + <script type="text/javascript"> + var applet = document.getElementById('AppletId'); + function resizeApplet(dim) { + applet.style.height = dim + "px"; + applet.style.width = dim + "px"; + } + for (var i = 2; i <= 5; i++) { + applet.print("Resizing to " + (i * 100) + " by " + (i * 100)); + resizeApplet(i * 100); + applet.sleep(50); + } + applet.print("*** APPLET FINISHED ***"); + </script> + +</body> +</html>
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/tests/reproducers/simple/ResizeApplet/srcs/ResizeApplet.java Tue May 21 09:31:57 2013 -0400 @@ -0,0 +1,72 @@ + +import java.applet.Applet; + +/* AppletTest.java +Copyright (C) 2011 Red Hat, Inc. + +This file is part of IcedTea. + +IcedTea is free software; you can redistribute it and/or +modify it under the terms of the GNU General Public License as published by +the Free Software Foundation, version 2. + +IcedTea is distributed in the hope that it will be useful, +but WITHOUT ANY WARRANTY; without even the implied warranty of +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +General Public License for more details. + +You should have received a copy of the GNU General Public License +along with IcedTea; see the file COPYING. If not, write to +the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA +02110-1301 USA. + +Linking this library statically or dynamically with other modules is +making a combined work based on this library. Thus, the terms and +conditions of the GNU General Public License cover the whole +combination. + +As a special exception, the copyright holders of this library give you +permission to link this library with independent modules to produce an +executable, regardless of the license terms of these independent +modules, and to copy and distribute the resulting executable under +terms of your choice, provided that you also meet, for each linked +independent module, the terms and conditions of the license of that +module. An independent module is a module which is not derived from +or based on this library. If you modify this library, you may extend +this exception to your version of the library, but you are not +obligated to do so. If you do not wish to do so, delete this +exception statement from your version. + */ +public class ResizeApplet extends Applet { + + /* Make sures the process is exited if we stall */ + private static class StallTimeoutThread extends Thread { + private static final int MILLISECONDS_TO_SLEEP = 5000; + @Override + public void run() { + try { + Thread.sleep(MILLISECONDS_TO_SLEEP); + System.out.println("*** APPLET FINISHED ***"); + } catch (InterruptedException ie) { + } + } + } + + @Override + public void init() { + new StallTimeoutThread().start(); + } + + /* Utility for Javascript-side */ + public void print(String str) { + System.out.println(str); + } + + /* Utility for Javascript-side */ + public synchronized void sleep(int time) { + try { + wait(time); + } catch (InterruptedException e) { + } + } +}
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/tests/reproducers/simple/ResizeApplet/testcases/ResizeAppletTests.java Tue May 21 09:31:57 2013 -0400 @@ -0,0 +1,65 @@ +/* AppletTestTests.java +Copyright (C) 2011 Red Hat, Inc. + +This file is part of IcedTea. + +IcedTea is free software; you can redistribute it and/or +modify it under the terms of the GNU General Public License as published by +the Free Software Foundation, version 2. + +IcedTea is distributed in the hope that it will be useful, +but WITHOUT ANY WARRANTY; without even the implied warranty of +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +General Public License for more details. + +You should have received a copy of the GNU General Public License +along with IcedTea; see the file COPYING. If not, write to +the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA +02110-1301 USA. + +Linking this library statically or dynamically with other modules is +making a combined work based on this library. Thus, the terms and +conditions of the GNU General Public License cover the whole +combination. + +As a special exception, the copyright holders of this library give you +permission to link this library with independent modules to produce an +executable, regardless of the license terms of these independent +modules, and to copy and distribute the resulting executable under +terms of your choice, provided that you also meet, for each linked +independent module, the terms and conditions of the license of that +module. An independent module is a module which is not derived from +or based on this library. If you modify this library, you may extend +this exception to your version of the library, but you are not +obligated to do so. If you do not wish to do so, delete this +exception statement from your version. + */ + +import static org.junit.Assert.assertTrue; + +import net.sourceforge.jnlp.ProcessResult; +import net.sourceforge.jnlp.ServerAccess.AutoClose; +import net.sourceforge.jnlp.browsertesting.BrowserTest; +import net.sourceforge.jnlp.browsertesting.Browsers; +import net.sourceforge.jnlp.closinglisteners.AutoOkClosingListener; +import net.sourceforge.jnlp.annotations.NeedsDisplay; +import net.sourceforge.jnlp.annotations.TestInBrowsers; + +import org.junit.Test; + +public class ResizeAppletTests extends BrowserTest { + + void assertContains(String source, String message, String substring) { + assertTrue(source + " should contain '" + substring + "' but did not!", + message.contains(substring)); + } + + @Test + @TestInBrowsers(testIn = { Browsers.all }) + @NeedsDisplay + public void testResizing() throws Exception { + ProcessResult pr = server.executeBrowser("/ResizeApplet.html", AutoClose.CLOSE_ON_CORRECT_END); + assertContains("stdout", pr.stdout, AutoOkClosingListener.MAGICAL_OK_CLOSING_STRING); + assertContains("stdout", pr.stdout, "Resizing to 500 by 500"); + } +}