Sergey (B),
I assume you updated the test with the bug ID ?
That's the normal convention as a test uncovers a regression to indicate
which test to run to verify the fix.

In this case the failing test is in "closed" so I can't see it here to verify you did this
I know we have a backlog of such tests still to move
Perhaps if you'd moved it to "open" at the same time it would be easier.

-phil.

On 2/22/2013 8:14 AM, Sergey Bylokhov wrote:
Hi, Sergey.
The bug(7194902) was found by the test.

22.02.2013 19:47, Sergey Malenkov wrote:
The fix looks good. Is it possible to create a test to ensure that the correct button is used?

Thanks,
SAM

On 13.02.2013 18:42, Sergey Bylokhov wrote:
cc swing-dev

Hello,
Please review the fix for jdk 8.
convertMouseEvent should use correct button info instead of NOBUTTON +
small cleanup.
Bugs:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7194902
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7181403
Webrev can be found at:
http://cr.openjdk.java.net/~serb/7194902/webrev.00/






Reply via email to