<AWT Dev> Review Request for 8041915: Move 8 awt tests to OpenJDK regression tests tree

[Alexander Scherbatiy]

   The fix looks good for me.

   Thanks,
   Alexandr.

On 5/26/2014 1:41 PM, Dmitriy Ermashov wrote:
> Hi,
>
> I still have no second successful review..
> Could you please look at the fix of
> https://bugs.openjdk.java.net/browse/JDK-8041915
>
> Webrev is here:
> http://cr.openjdk.java.net/~yan/8041915/webrev.01/
>
> It is a part of test colocation.
>
> Thanks,
> Dima
>
> On 05/12/2014 03:16 PM, Dmitriy Ermashov wrote:
>> Petr, thanks for review.
>>
>> Guys, could you please also review the changeset?
>> Webrev is here:
>> http://cr.openjdk.java.net/~yan/8041915/webrev.01/
>>
>> Thanks,
>> Dima
>>
>> On 29.04.2014 15:08, Petr Pchelko wrote:
>>> Hello, Dmitriy.
>>>
>>> The new version looks good.
>>>
>>> With best regards. Petr.
>>>
>>> On 29.04.2014, at 14:49, Dmitriy Ermashov 
>>> <dmitriy.ermashov at oracle.com> wrote:
>>>
>>>> Hi,
>>>>
>>>> Please review the changeset for
>>>> https://bugs.openjdk.java.net/browse/JDK-8041915
>>>>
>>>> Webrev is here:
>>>> http://cr.openjdk.java.net/~yan/8041915/webrev.01/
>>>>
>>>> Latest changes:
>>>> 1. If some translucency mode is not supported, the test will pass 
>>>> with System.out warning message
>>>> 2. New method dragAndDrop implemented in ExtendedRobot class
>>>>
>>>> Thanks,
>>>> Dima
>>>>
>>>> On 04/25/2014 05:19 PM, Petr Pchelko wrote:
>>>>> Hello, Dmitriy.
>>>>>
>>>>> A couple of questions:
>>>>>
>>>>> 1. checkTranslucencyMode throws an exception if some mode is not 
>>>>> supported on the device, so the test would fail. Should it? 
>>>>> Normally we just skip the test if some capability is absent.
>>>>> 2. Didn't you consider moving the drag method into the 
>>>>> ExtendedRobot? I expect it to be very commonly used.
>>>>>
>>>>> With best regards. Petr.
>>>>>
>>>>> On 25.04.2014, at 17:04, Dmitriy Ermashov 
>>>>> <dmitriy.ermashov at oracle.com> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> Please review the changeset for
>>>>>> https://bugs.openjdk.java.net/browse/JDK-8041915
>>>>>>
>>>>>> Webrev is here:
>>>>>> http://cr.openjdk.java.net/~yan/8041915/webrev.00/
>>>>>>
>>>>>> -- 
>>>>>> Thanks,
>>>>>> Dima
>>>>>>
>>
>