Hi, Jay.
> I tried different failure scenarios inside test like making comparing
> images fail but all the times frames are getting disposed.
> Could you please elaborate in which scenario exactly you are seeing
> frame not getting disposed?

I run the version of the test before your fix, and it leaves one of the frame 
of the screen.

> 
> But I have still added explicit frame.dispose() at places to make sure
> that we dispose the created frames.
> Please find updated webrev for review:
> http://cr.openjdk.java.net/~jdv/8197926/webrev.01/ 
> 
> Also in continuation to what we discussed offline, I used
> -vmoptions:"-Dsun.java2d.uiScale=2" to pass to jtreg from command
> prompt. But in this case also System.getProperty("sun.java2d.uiScale")
> returned null in test case. So I was not able to pass uiScale
> mentioned in command prompt to ProcessCommunicator in test case.

I tested it using this command line:
../jtreg/bin/jtreg -vmoptions:"-Dsun.java2d.uiScale=10" -verbose:summary -a -nr 
-jdk:./build/windows-x64/images/jdk/  
open/test/jdk/java/awt/dnd/ImageTransferTest/ImageTransferTest.java

Plus this change in the test:
    public static void main(String[] arg) throws Exception {
+       String prop = System.getProperty("sun.java2d.uiScale");
+       System.err.println(prop);

It prints 10 in the log.

Reply via email to