There is no changeset in JBS for 6738668 (probably because it was before mercurial era) which could have told us what was the fix.

On 5/26/2017 10:22 AM, Prasanta Sadhukhan wrote:
What we should do with this test fix in linux?

Regards
Prasanta
On 5/25/2017 11:24 AM, Prasanta Sadhukhan wrote:
Hi Sergey,

I do not see any comment mention about failure in 7b20 but anyways, I tried with 7b20,7b30,7b40,7b50 and all of them passed with original and updated test in windows.

Regards
Prasanta
On 5/25/2017 2:30 AM, Sergey Bylokhov wrote:
I am not sure but according to the comments of JDK-6738668 it was a regression in 7b20.
So I suggest to check a few build between b20 and b54

----- prasanta.sadhuk...@oracle.com wrote:

Hi Sergey,

No, it is passing. Actually, the original test is also passing with
1.7.0 b05 (6738668 is supposedly fixed in 7b55), there's no
SecurityException

jdk1.7.0/bin/java -Djava.security.policy=security.policy bug6738668
Test passed for LookAndFeel javax.swing.plaf.metal.MetalLookAndFeel
Test passed for LookAndFeel
com.sun.java.swing.plaf.motif.MotifLookAndFeel
Test passed for LookAndFeel
com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Test passed for LookAndFeel
com.sun.java.swing.plaf.windows.WindowsClassicLookAndFeel

jdk1.7.0/bin/java -version
java version "1.7.0-ea"
Java(TM) SE Runtime Environment (build 1.7.0-ea-b05)
Java HotSpot(TM) 64-Bit Server VM (build 1.7.0-ea-b05, mixed mode)

Regards
Prasanta
On 5/24/2017 11:48 AM, Sergey Bylokhov wrote:
Hi, Prasanta.
Please confirm that the updated test fails before JDK-6738668 was
fixed.
Hi ALl,

Please review a testbug fix for an issue where this regression test
is failing in linux because
JFileChooser was trying to access C:/temp path which does not exist
in linux.
Proposed fix is to use java.io.tmpdir instead of hardcoded windows
path.
Bug: https://bugs.openjdk.java.net/browse/JDK-6962725
webrev: http://cr.openjdk.java.net/~psadhukhan/6962725/webrev.00/

Regards
Prasanta



Reply via email to