I do not think that the fix is correct.

SAM

On 24.07.2013 19:14, sergey malenkov wrote:
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


Reply via email to