<Swing Dev> [8] Review request for CR 8006087 [TEST_BUG] The BACKSPACE key doesn't work and after pressing 'cancel' and 'DONE' button, the case pass automatically.

[sergey malenkov]

Hi Konstantin,

Why you moved the JavaTest header to the HTML file?
Almost all regression tests use a Java file for it.
I think you should leave it "as is".

Note that other HTML files in the test/javax/swing folder
does not have the Oracle Copyright text.
I'm not sure that we should add it into this file only.

Other changes in your fix looks good for me.

Thanks,
SAM

On 19.06.2013 14:01, Konstantin Shefov wrote:
> Hello,
>
> Please review a fix for the issue:
>
> 8006087 [TEST_BUG] The BACKSPACE key doesn't work and after pressing 
> 'cancel' and 'DONE' button, the case pass automatically.
>
> Test bug fix.
>
> http://bugs.sun.com/view_bug.do?bug_id=8006087
>
> The webrev is: http://cr.openjdk.java.net/~kshefov/8006087/webrev.00
>
> Thanks,
> Konstantin