Hello, Anthony. The fix look good.
With best regards. Petr. On 16.12.2013, at 18:22, Anthony Petrov <[email protected]> wrote: > Hi Petr, > > Thank you for the review. Please find a webrev with a new regression test > included at: > > http://cr.openjdk.java.net/~anthony/8-3-acceptDrop-8029979.1/ > > -- > best regards, > Anthony > > On 12/14/2013 03:07 AM, Petr Pchelko wrote: >> Hello, Anthony. >> >> The code changes look good, but could you please add a reg test for this >> issue. >> >> With best regards. Petr. >> >> On Dec 13, 2013, at 8:53 PM, Anthony Petrov <[email protected]> >> wrote: >> >>> Hi Petr, Sergey, >>> >>> Please review a fix for https://bugs.openjdk.java.net/browse/JDK-8029979 at: >>> >>> http://cr.openjdk.java.net/~anthony/8-3-acceptDrop-8029979.0/ >>> >>> I enable calling SunDropTargetContextPeer.acceptDrop() as many times as >>> needed, for as long as the DnD operation isn't complete yet. Running open >>> and closed DnD regression tests revealed no new failures. >>> >>> Note that later we will need to back-port this fix to 8u20. >>> >>> -- >>> best regards, >>> Anthony >>
