<AWT Dev> [9] Review Request for 8054359: move awt automated tests from AWT_Modality to OpenJDK repository - part 8

[alexander stepanov]

Hello,

Could please anyone review this old request?

Thanks,
Alexander

On 25.09.2014 17:08, alexander stepanov wrote:
> Sorry, just a reminder.
>
> Thanks,
> Alexander
>
> On 12.09.2014 15:08, alexander stepanov wrote:
>> Hello Alexandr,
>>
>> Fixed; please see the updated webrev:
>> http://cr.openjdk.java.net/~avstepan/8054359/webrev.01/
>>
>> Thanks,
>> Alexander
>>
>> On 12.09.2014 12:49, Alexander Scherbatiy wrote:
>>>
>>>   75         int x = 
>>> Toolkit.getDefaultToolkit().getScreenSize().width - 200;
>>>   76         parent.setLocation(x, 50);
>>>   77         window.setLocation(x, 250);
>>>   78         dialog.setLocation(x, 450);
>>>
>>>  It is better to use part of the screen or frame size instead of 
>>> absolute values like 200, 50 and 450.
>>>
>>>  Thanks,
>>>  Alexandr.
>>>
>>> On 9/11/2014 6:31 PM, alexander stepanov wrote:
>>>> Sorry, just a reminder. Could anyone please review that?
>>>>
>>>> Thanks,
>>>> Alexander
>>>>
>>>>
>>>> On 22.08.2014 20:17, alexander stepanov wrote:
>>>>> Hello,
>>>>>
>>>>> Could you please review the fix for
>>>>> https://bugs.openjdk.java.net/browse/JDK-8054359
>>>>>
>>>>> webrev:
>>>>> http://cr.openjdk.java.net/~avstepan/8054359/
>>>>>
>>>>> This is the next portion of functional AWT tests prepared for 
>>>>> migration to OpenJDK repository.
>>>>>
>>>>> The tests were checked on Ubuntu 14.04 Linux, Solaris 11 and 
>>>>> Windows 7.
>>>>>
>>>>> On Mac OS X 10.8.5 they are failing due to unfixed
>>>>> https://bugs.openjdk.java.net/browse/JDK-7186009
>>>>>
>>>>> Some of them are also failing because of
>>>>> https://bugs.openjdk.java.net/browse/JDK-8055752
>>>>>
>>>>> Thanks,
>>>>> Alexander
>>>>
>>>
>>
>