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
