<Swing Dev> Fix for JDK-8065861 : Pressing Esc does not set 'canceled' property of ProgressMonitor

Alexandr Scherbatiy alexandr.scherbatiy at oracle.com
Wed Jun 22 12:32:36 UTC 2016


The fix looks good to me.

Thanks,
Alexandr.

On 6/22/2016 3:26 PM, Ajit Ghaisas wrote:
> Hi,
>
>      There was a suggestion to modify the Javadoc text.
>      The code changes stand unchanged from webrev.01.   Only Javadoc comment has been modified.
>
>       Request you to review again :
>       http://cr.openjdk.java.net/~aghaisas/8065861/webrev.02/
>      
>
> Regards,
> Ajit
>
> -----Original Message-----
> From: Rajeev Chamyal
> Sent: Monday, June 13, 2016 1:47 PM
> To: Alexander Scherbatiy; Ajit Ghaisas; Sergey Bylokhov; swing-dev at openjdk.java.net
> Subject: RE: Fix for JDK-8065861 : Pressing Esc does not set 'canceled' property of ProgressMonitor
>
> Looks good to me.
>
> Regards,
> Rajeev Chamyal
>
> -----Original Message-----
> From: Alexandr Scherbatiy
> Sent: 10 June 2016 13:06
> To: Ajit Ghaisas; Sergey Bylokhov; Rajeev Chamyal; swing-dev at openjdk.java.net
> Subject: Re: Fix for JDK-8065861 : Pressing Esc does not set 'canceled' property of ProgressMonitor
>
>
> The fix looks good to me.
>
> Thanks,
> Alexandr.
>
> On 6/10/2016 8:50 AM, Ajit Ghaisas wrote:
>> Hi,
>>
>>       Thanks Alex for spotting probable exceptions in code changes.
>>        I have corrected the code to address them.
>>
>>        Here is the updated webrev. Request you to review.
>>        http://cr.openjdk.java.net/~aghaisas/8065861/webrev.01/
>>
>> Regards,
>> Ajit
>>      
>>
>> -----Original Message-----
>> From: Alexandr Scherbatiy
>> Sent: Thursday, June 09, 2016 8:56 PM
>> To: Ajit Ghaisas; Sergey Bylokhov; Rajeev Chamyal; swing-dev at openjdk.java.net
>> Subject: Re: Fix for JDK-8065861 : Pressing Esc does not set 'canceled' property of ProgressMonitor
>>
>> On 6/8/2016 5:35 PM, Ajit Ghaisas wrote:
>>> Hi,
>>>
>>> Bug :
>>> https://bugs.openjdk.java.net/browse/JDK-8065861
>>>
>>> Issue :
>>> Pressing Esc does not set 'canceled' property of ProgressMonitor
>>>
>>> Analysis :
>>> ProgressMonitor option pane only gets hidden on pressing Escape key.
>>> It is not truly canceled as isCanceled() method continues to return false.
>>>
>>> Fix :
>>> On Pressing Escape key JOptionPane.CLOSED_OPTION value is set in JOptionPane from BasicOptionPaneUI.java.
>>> This value is used as a condition in isCanceled() to identify ProgressMonitor is canceled by pressing Escape key.
>>>
>>> Please review the webrev:
>>> http://cr.openjdk.java.net/~aghaisas/8065861/webrev.00/
>>         The updated code calls methods from 'v' object before it is checked to the null which can lead to the NPE. The cancelOption[0] also can throw the IOBE before the arrays length checking.
>>
>>      Thanks,
>>      Alexandr.
>>> On review completion, I will raise a CCC request for documentation change.
>>>
>>> Regards,
>>> Ajit




More information about the swing-dev mailing list