<AWT Dev> Request for review: 7155298 : Editable TextArea blocks GUI application from exit
zhouyx at linux.vnet.ibm.com
Wed Mar 21 05:07:17 PDT 2012
Thanks for the suggestion, I didn't notice that. The timeout would
simplify the testcase much, I'll rewrite it.
On Wed, Mar 21, 2012 at 7:36 PM, Anthony Petrov
<anthony.petrov at oracle.com>wrote:
> Hi Sean,
> The code changes look fine to me. However, I see an issue with the test.
> It seems to verify a concrete implementation rather than the expected
> Since disposing a text area prevents an app from exiting w/o your fix,
> perhaps exactly this needs to be tested? We don't want to use reflection to
> access specific fields in peer classes, and then verify their internal
> state. Actually, after calling dispose() and returning from main() a jtreg
> test would simply fail by time out, and that would indicate a regression.
> With your fix applied, however, the test would correctly exit, and thus
> best regards,
> On 3/21/2012 11:55 AM, Sean Chou wrote:
>> Hi all,
>> It is found that when TextArea or TextField is setEditable(true), the
>> application will not exit after the application frame disposes.
>> The reason is the visible caret contains a timer which controls its
>> flashing . This timer must be stopped when disposing.
>> Link to webrev: http://cr.openjdk.java.net/~**zhouyx/7155298/webrev.00/<http://cr.openjdk.java.net/%7Ezhouyx/7155298/webrev.00/> Link to bug:
>> Best Regards,
>> Sean Chou
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the awt-dev