<AWT Dev> [9] Review Request for 8132958: [TEST_BUG] Part 1: update client tests failing after changes in setAccessible(true) routine
Alexander Scherbatiy
alexandr.scherbatiy at oracle.com
Thu Aug 13 10:07:27 UTC 2015
The fix looks good to me.
Thanks,
Alexandr.
On 8/13/2015 10:54 AM, Yuri Nesterenko wrote:
> Please see the second version:
> http://cr.openjdk.java.net/~yan/8132958/webrev.01/
> Test test/java/awt/image/MultiResolutionImageTest.java removed from
> the fix.
>
> Reference webrev about a moved test is unchanged:
> http://cr.openjdk.java.net/~yan/8132958/webrev.diff.00/
>
> Thanks,
> -yan
>
> On 08/12/2015 06:33 PM, Alexander Scherbatiy wrote:
>>
>> The test/java/awt/image/MultiResolutionImageTest.java has part which
>> can be run on all platforms.
>>
>> There is the specific check for Mac OS X dependent part. It looks like
>> the test does not need to have the @requires mac tag.
>>
>> Thanks,
>> Alexandr.
>>
>> On 8/12/2015 4:29 PM, Yuri Nesterenko wrote:
>>> Colleagues,
>>>
>>> please review this test-only fix.
>>> Some tests would need more elaborate permissions now to access
>>> the private methods of internal classes.
>>>
>>> CR is https://bugs.openjdk.java.net/browse/JDK-8132958
>>> Webrev see in http://cr.openjdk.java.net/~yan/8132958/webrev.00/
>>> A diff in a moved closed test is in
>>> http://cr.openjdk.java.net/~yan/8132958/webrev.diff.00/
>>>
>>> Thanks,
>>> -yan
>>
>
More information about the awt-dev
mailing list