OpenJDK / jdk / hs
changeset 12285:37f0c586ba99
7155298: Editable TextArea/TextField are blocking GUI applications from exit
Summary: Stop default caret's timer by setVisible(false) when dispose
Reviewed-by: anthony, ant
author | zhouyx |
---|---|
date | Sat, 31 Mar 2012 16:55:56 +0800 |
parents | 3f2477ebb708 |
children | 9e40486e227e |
files | jdk/src/solaris/classes/sun/awt/X11/XTextAreaPeer.java jdk/src/solaris/classes/sun/awt/X11/XTextFieldPeer.java jdk/test/java/awt/TextArea/DisposeTest/TestDispose.java jdk/test/java/awt/TextField/DisposeTest/TestDispose.java |
diffstat | 4 files changed, 207 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/jdk/src/solaris/classes/sun/awt/X11/XTextAreaPeer.java Fri Mar 30 18:10:05 2012 +0400 +++ b/jdk/src/solaris/classes/sun/awt/X11/XTextAreaPeer.java Sat Mar 31 16:55:56 2012 +0800 @@ -168,6 +168,8 @@ public void dispose() { XToolkit.specialPeerMap.remove(jtext); + // visible caret has a timer thread which must be stopped + jtext.getCaret().setVisible(false); jtext.removeNotify(); textPane.removeNotify(); super.dispose();
--- a/jdk/src/solaris/classes/sun/awt/X11/XTextFieldPeer.java Fri Mar 30 18:10:05 2012 +0400 +++ b/jdk/src/solaris/classes/sun/awt/X11/XTextFieldPeer.java Sat Mar 31 16:55:56 2012 +0800 @@ -104,6 +104,8 @@ public void dispose() { XToolkit.specialPeerMap.remove(xtext); + // visible caret has a timer thread which must be stopped + xtext.getCaret().setVisible(false); xtext.removeNotify(); super.dispose(); }
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/jdk/test/java/awt/TextArea/DisposeTest/TestDispose.java Sat Mar 31 16:55:56 2012 +0800 @@ -0,0 +1,101 @@ +/* + * Copyright (c) 2012 Oracle and/or its affiliates. All rights reserved. + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * + * This code is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 only, as + * published by the Free Software Foundation. + * + * This code 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 + * version 2 for more details (a copy is included in the LICENSE file that + * accompanied this code). + * + * You should have received a copy of the GNU General Public License version + * 2 along with this work; if not, write to the Free Software Foundation, + * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. + * + * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA + * or visit www.oracle.com if you need additional information or have any + * questions. + */ + +/* + * Portions Copyright (c) 2012 IBM Corporation + */ + +/* @test + * @bug 7155298 + * @run main/othervm/timeout=60 TestDispose + * @summary Editable TextArea blocks GUI application from exit. + * @author Sean Chou + */ + +import java.awt.FlowLayout; +import java.awt.Frame; +import java.awt.TextArea; +import java.awt.Toolkit; +import java.lang.reflect.InvocationTargetException; + +import javax.swing.JFrame; +import javax.swing.SwingUtilities; + +import sun.awt.SunToolkit; + +public class TestDispose { + + public static Frame frame = null; + public static TextArea textArea = null; + public static volatile Process worker = null; + + public void testDispose() throws InvocationTargetException, + InterruptedException { + SunToolkit toolkit = (SunToolkit) Toolkit.getDefaultToolkit(); + + SwingUtilities.invokeAndWait(new Runnable() { + @Override + public void run() { + frame = new JFrame("Test"); + + textArea = new TextArea("editable textArea"); + textArea.setEditable(true); + // textArea.setEditable(false); // this testcase passes if textArea is non-editable + + frame.setLayout(new FlowLayout()); + frame.add(textArea); + + frame.pack(); + frame.setVisible(true); + } + }); + toolkit.realSync(); + + SwingUtilities.invokeAndWait(new Runnable() { + @Override + public void run() { + frame.dispose(); + } + }); + toolkit.realSync(); + } + + public static void main(String[] args) throws Exception{ + if(args.length == 0) { + Runtime.getRuntime().addShutdownHook(new Thread(){ + public void run() { + worker.destroy(); + } + }); + + System.out.println(System.getProperty("java.home")+"/bin/java TestDispose workprocess"); + worker = Runtime.getRuntime().exec(System.getProperty("java.home")+"/bin/java TestDispose workprocess"); + worker.waitFor(); + return; + } + + TestDispose app = new TestDispose(); + app.testDispose(); + } + +}
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/jdk/test/java/awt/TextField/DisposeTest/TestDispose.java Sat Mar 31 16:55:56 2012 +0800 @@ -0,0 +1,102 @@ +/* + * Copyright (c) 2012 Oracle and/or its affiliates. All rights reserved. + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * + * This code is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 only, as + * published by the Free Software Foundation. + * + * This code 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 + * version 2 for more details (a copy is included in the LICENSE file that + * accompanied this code). + * + * You should have received a copy of the GNU General Public License version + * 2 along with this work; if not, write to the Free Software Foundation, + * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. + * + * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA + * or visit www.oracle.com if you need additional information or have any + * questions. + */ + +/* + * Portions Copyright (c) 2012 IBM Corporation + */ + +/* @test + * @bug 7155298 + * @run main/othervm/timeout=60 TestDispose + * @summary Editable TextField blocks GUI application from exit. + * @author Sean Chou + */ + +import java.awt.FlowLayout; +import java.awt.Frame; +import java.awt.TextField; +import java.awt.Toolkit; +import java.lang.reflect.InvocationTargetException; + +import javax.swing.JFrame; +import javax.swing.SwingUtilities; + +import sun.awt.SunToolkit; + +public class TestDispose { + + public static Frame frame = null; + public static TextField textField = null; + public static volatile Process worker = null; + + public void testDispose() throws InvocationTargetException, + InterruptedException { + SunToolkit toolkit = (SunToolkit) Toolkit.getDefaultToolkit(); + + SwingUtilities.invokeAndWait(new Runnable() { + @Override + public void run() { + frame = new JFrame("Test"); + + textField = new TextField("editable textArea"); + textField.setEditable(true); + // textField.setEditable(false); // this testcase passes if textField is non-editable + + frame.setLayout(new FlowLayout()); + frame.add(textField); + + frame.pack(); + frame.setVisible(true); + } + }); + toolkit.realSync(); + + SwingUtilities.invokeAndWait(new Runnable() { + @Override + public void run() { + frame.dispose(); + } + }); + toolkit.realSync(); + + } + + public static void main(String[] args) throws Exception{ + if(args.length == 0) { + Runtime.getRuntime().addShutdownHook(new Thread(){ + public void run() { + worker.destroy(); + } + }); + + System.out.println(System.getProperty("java.home")+"/bin/java TestDispose workprocess"); + worker = Runtime.getRuntime().exec(System.getProperty("java.home")+"/bin/java TestDispose workprocess"); + worker.waitFor(); + return; + } + + TestDispose app = new TestDispose(); + app.testDispose(); + } + +}