On Wed, 20 Mar 2024 16:59:47 GMT, Harshitha Onkar <hon...@openjdk.org> wrote:

>> Following test is converted from applet to main using PassFailJFrame.
>> 
>> The original test had ignore tag - **The fix for this problem was backed out 
>> for 5057184.  Remove this "ignore" tag when it is fixed again.**
>> I believe this was fixed and the test works correctly on Linux platforms, 
>> hence ignore tag has been removed.
>> 
>> This is a semi-automated test, the first check is manual and the 2nd one is 
>> automated. Details in the test instructions.
>
> Harshitha Onkar has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   instruction line length

test/jdk/java/awt/im/bug4490692.java line 81:

> 79: 
> 80: class TestFrame extends JFrame implements KeyListener {
> 81:     JTextField text;

text can be moved inside constructor, any reason why it is here?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18402#discussion_r1533639696

Reply via email to