<AWT Dev> Review Request for 8041915: Move 8 awt tests to OpenJDK regression tests tree
Dmitriy Ermashov
dmitriy.ermashov at oracle.com
Mon May 26 09:41:29 UTC 2014
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
>>>>>
>
More information about the awt-dev
mailing list