<AWT Dev> [7] Review request for 6891269: non-compliance with Section 508 using 1.6.0_18

[Vladislav Karnaukhov]

On 04.06.12 17:48, Artem Ananiev wrote:
> Hi, Vladislav,
>
> some random comments:
>
> 1. From the bug description, it's not clear when the problem is 
> reproducible. Does caret size become 1x1 pixels on windows theme changes?
The problem is reproducible when caret width is changed in Windows 
Accessibility/Ease Of Access tool. AwtTextArea doesn't set new caret 
width, it's always default width (1 pixel).
>
>
> 2. AwtComponent class already has WmSettingsChanged() callback, so you 
> can place ::SystemParametersInfo() there, not into AwtToolkit.
See below.
>
> 3. Does the problem only manifests in text areas? What about text fields?
Text fields redraw caret correctly. Only text areas seem to have this 
defect.
>
> 4. Could you describe why UpdateCaret() is required in WmForwardChar() 
> and in HandleEvent()? I would expect ::CreateCaret() in WmSetFocus(), 
> ::DestroyCaret() in WmKillFocus(), and re-create caret in 
> WmSettingChanged() should be enough.
>
> 5. If we introduce UpdateCaret() method, it should be placed either to 
> AwtTextArea or to AwtTextComponent, there is no need to have this 
> method in AwtComponent.
I'll reconsider the fix keeping in mind your comments. Will submit new 
review request.

Regards,
- Vlad
>
>
> Thanks,
>
> Artem
>
> On 6/1/2012 6:28 PM, Vladislav Karnaukhov wrote:
>> Hello,
>>
>> please review a fix for the following issue: 6891269 non-compliance with
>> Section 508 using 1.6.0_18
>>
>> http://monaco.sfbay.sun.com/detail.jsf?cr=6891269
>>
>> Please find webrev here:
>> http://cr.openjdk.java.net/~alexp/6891269/webrev.00/
>>
>> This is a forward-port from JDK6. The fix introduces support for Windows
>> Accessibility/Ease Of Access to TextArea control.
>>
>> Regards,
>> - Vlad