NEWT/ScreenMode: Fix Bug 456 - Reset to orig ScreenMode when exit (normal or ctrl-c) ; Generics Coding

7 files changed, 209 insertions, 48 deletions

diff --git a/make/scripts/tests-x64.bat b/make/scripts/tests-x64.bat
index b3edb11d9..046d2c68d 100755
--- a/make/scripts/tests-x64.bat
+++ b/make/scripts/tests-x64.bat
@@ -35,6 +35,7 @@ REM scripts\java-win64.bat com.jogamp.opengl.test.junit.newt.TestFocus02SwingAWT
 REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.newt.TestScreenMode00NEWT
 REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.newt.TestScreenMode01NEWT
 REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.newt.TestScreenMode02NEWT
+scripts\java-win64.bat com.jogamp.opengl.test.junit.newt.ManualScreenMode03NEWT
 
 REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.newt.TestDisplayLifecycle01NEWT
 REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.newt.parenting.TestParenting01NEWT
@@ -61,7 +62,7 @@ REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.jogl.offscreen.TestO
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.graph.TestRegionRendererNEWT01
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.graph.TestTextRendererNEWT01
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.jogl.glsl.TestGLSLShaderState02NEWT
-scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.jogl.glsl.TestGLSLShaderState02NEWT
+REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.jogl.glsl.TestGLSLShaderState02NEWT
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.graph.demos.ui.UINewtDemo01
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.graph.demos.GPUTextNewtDemo01
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.graph.demos.GPUTextNewtDemo02
@@ -72,7 +73,7 @@ REM scripts\java-win64.bat com.jogamp.opengl.test.junit.jogl.acore.TestGLDebug00
 REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.jogl.acore.TestGLDebug01NEWT $*
 
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.jogl.acore.TestGPUMemSec01NEWT $*
-scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.jogl.acore.TestGPUMemSec01NEWT $*
+REM scripts\java-win64-dbg.bat com.jogamp.opengl.test.junit.jogl.acore.TestGPUMemSec01NEWT $*
 
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.jogl.glsl.TestRulerNEWT01
 REM scripts\java-win64.bat com.jogamp.opengl.test.junit.jogl.glsl.TestFBOMRTNEWT01
diff --git a/make/scripts/tests.sh b/make/scripts/tests.sh
index 403ea86c6..5ba1c6e0e 100755
--- a/make/scripts/tests.sh
+++ b/make/scripts/tests.sh
@@ -143,7 +143,7 @@ function testawtmt() {
 #testawt com.jogamp.opengl.test.junit.jogl.demos.gl2.newt.TestGearsNEWT $*
 #testnoawt com.jogamp.opengl.test.junit.jogl.demos.gl2.newt.TestGearsNEWT $*
 #testnoawt com.jogamp.opengl.test.junit.jogl.demos.es1.newt.TestGearsES1NEWT $*
-testnoawt com.jogamp.opengl.test.junit.jogl.demos.es2.newt.TestGearsES2NEWT $*
+#testnoawt com.jogamp.opengl.test.junit.jogl.demos.es2.newt.TestGearsES2NEWT $*
 #testnoawt com.jogamp.opengl.test.junit.newt.TestGLWindows02NEWTAnimated $*
 #testawt com.jogamp.opengl.test.junit.newt.TestDisplayLifecycle01NEWT
 #testawt com.jogamp.opengl.test.junit.newt.TestDisplayLifecycle02NEWT
@@ -152,6 +152,7 @@ testnoawt com.jogamp.opengl.test.junit.jogl.demos.es2.newt.TestGearsES2NEWT $*
 #testawt com.jogamp.opengl.test.junit.newt.TestScreenMode00NEWT
 #testnoawt com.jogamp.opengl.test.junit.newt.TestScreenMode01NEWT
 #testnoawt com.jogamp.opengl.test.junit.newt.TestScreenMode02NEWT
+testnoawt com.jogamp.opengl.test.junit.newt.ManualScreenMode03NEWT
 #testawt com.jogamp.opengl.test.junit.newt.TestGLWindows01NEWT -time 1000000
 #testawt -Djava.awt.headless=true com.jogamp.opengl.test.junit.newt.TestGLWindows01NEWT
 #testnoawt com.jogamp.opengl.test.junit.jogl.swt.TestSWT01GLn $*
diff --git a/src/newt/classes/jogamp/newt/ScreenImpl.java b/src/newt/classes/jogamp/newt/ScreenImpl.java
index cc0d03e01..59d46a662 100644
--- a/src/newt/classes/jogamp/newt/ScreenImpl.java
+++ b/src/newt/classes/jogamp/newt/ScreenImpl.java
@@ -41,9 +41,12 @@ import com.jogamp.newt.NewtFactory;
 import com.jogamp.newt.Screen;
 import com.jogamp.newt.ScreenMode;
 import com.jogamp.newt.event.ScreenModeListener;
+import com.jogamp.newt.util.MonitorMode;
 import com.jogamp.newt.util.ScreenModeUtil;
 
 import javax.media.nativewindow.*;
+import javax.media.nativewindow.util.DimensionImmutable;
+import javax.media.nativewindow.util.SurfaceSize;
 
 import java.security.*;
 import java.util.ArrayList;
@@ -61,13 +64,21 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
     protected int width=-1, height=-1; // detected values: set using setScreenSize
     protected static int usrWidth=-1, usrHeight=-1; // property values: newt.ws.swidth and newt.ws.sheight
     private static AccessControlContext localACC = AccessController.getContext();
-    private List/*<ScreenModeListener>*/ referencedScreenModeListener = new ArrayList();
+    private ArrayList<ScreenModeListener> referencedScreenModeListener = new ArrayList<ScreenModeListener>();
     long t0; // creationTime
 
-    private static Class getScreenClass(String type) 
-    throws ClassNotFoundException 
+    static {
+        AccessController.doPrivileged(new PrivilegedAction() {
+            public Object run() {
+                registerShutdownHook();
+                return null;
+            }
+        });
+    }
+    
+    private static Class<? extends Screen> getScreenClass(String type) throws ClassNotFoundException 
     {
-        Class screenClass = NewtFactory.getCustomClass(type, "Screen");
+        Class<?> screenClass = NewtFactory.getCustomClass(type, "Screen");
         if(null==screenClass) {
             if (NativeWindowFactory.TYPE_ANDROID.equals(type)) {
                 screenClass = Class.forName("jogamp.newt.driver.android.AndroidScreen");
@@ -85,7 +96,7 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
                 throw new RuntimeException("Unknown window type \"" + type + "\"");
             }
         }
-        return screenClass;
+        return (Class<? extends Screen>)screenClass;
     }
 
     public static Screen create(Display display, final int idx) {
@@ -111,7 +122,7 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
                         return screen0;
                     }
                 }
-                Class screenClass = getScreenClass(display.getType());
+                Class<? extends Screen> screenClass = getScreenClass(display.getType());
                 ScreenImpl screen  = (ScreenImpl) screenClass.newInstance();
                 screen.display = (DisplayImpl) display;
                 screen.screen_idx = idx;
@@ -278,8 +289,8 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
         return "NEWT-Screen["+getFQName()+", idx "+screen_idx+", refCount "+refCount+", "+getWidth()+"x"+getHeight()+", "+aScreen+", "+display+"]";
     }
 
-    public final List/*<ScreenMode>*/ getScreenModes() {
-        ArrayHashSet screenModes = getScreenModesOrig();
+    public final List<ScreenMode> getScreenModes() {
+        ArrayHashSet<ScreenMode> screenModes = getScreenModesOrig();
         if(null != screenModes && 0 < screenModes.size()) {
             return screenModes.toArrayList();
         }
@@ -375,7 +386,7 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
     }
 
     /** ScreenModeStatus bridge to native implementation */
-    protected final ArrayHashSet getScreenModesOrig() {
+    protected final ArrayHashSet<ScreenMode> getScreenModesOrig() {
         ScreenModeStatus sms = ScreenModeStatus.getScreenModeStatus(this.getFQName());
         if(null!=sms) {
             return sms.getScreenModes();
@@ -437,6 +448,7 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
     }
 
     private void initScreenModeStatus() {
+        // JAU: FIXME: Add return ..
         ScreenModeStatus sms;
         ScreenModeStatus.lockScreenModeStatus();
         try {
@@ -444,7 +456,7 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
             if(null==sms) {                
                 IntIntHashMap screenModesIdx2NativeIdx = new IntIntHashMap();
 
-                ArrayHashSet screenModes = collectNativeScreenModes(screenModesIdx2NativeIdx);
+                ArrayHashSet<ScreenMode> screenModes = collectNativeScreenModes(screenModesIdx2NativeIdx);
                 if(screenModes.size()==0) {
                     ScreenMode sm0 = ( DEBUG_TEST_SCREENMODE_DISABLED ) ? null : getCurrentScreenModeImpl();
                     if(null != sm0) {
@@ -476,14 +488,12 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
     }
 
     /** ignores bpp < 15 */
-    private ArrayHashSet collectNativeScreenModes(IntIntHashMap screenModesIdx2NativeId) {
-        ArrayHashSet resolutionPool  = new ArrayHashSet();
-        ArrayHashSet surfaceSizePool = new ArrayHashSet();
-        ArrayHashSet screenSizeMMPool = new ArrayHashSet();
-        ArrayHashSet monitorModePool = new ArrayHashSet();
-        ArrayHashSet screenModePool = null;
-
-        screenModePool = new ArrayHashSet();
+    private ArrayHashSet<ScreenMode> collectNativeScreenModes(IntIntHashMap screenModesIdx2NativeId) {
+        ArrayHashSet<DimensionImmutable> resolutionPool   = new ArrayHashSet<DimensionImmutable>();
+        ArrayHashSet<SurfaceSize>        surfaceSizePool  = new ArrayHashSet<SurfaceSize>();
+        ArrayHashSet<DimensionImmutable> screenSizeMMPool = new ArrayHashSet<DimensionImmutable>();
+        ArrayHashSet<MonitorMode>        monitorModePool  = new ArrayHashSet<MonitorMode>();
+        ArrayHashSet<ScreenMode>         screenModePool   = new ArrayHashSet<ScreenMode>();
 
         int[] smProps = null;
         int num = 0;
@@ -526,7 +536,7 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
         ScreenModeStatus sms;
         ScreenModeStatus.lockScreenModeStatus();
         try {
-            sms = ScreenModeStatus.getScreenModeStatus(this.getFQName());
+            sms = ScreenModeStatus.getScreenModeStatus(getFQName());
             if(null != sms) {
                 sms.lock();
                 try {
@@ -534,7 +544,7 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
                         if(!sms.isOriginalMode()) {
                             setCurrentScreenMode(sms.getOriginalScreenMode());
                         }
-                        ScreenModeStatus.unmapScreenModeStatus(this.getFQName());
+                        ScreenModeStatus.unmapScreenModeStatus(getFQName());
                     }
                 } finally {
                     sms.unlock();
@@ -544,5 +554,38 @@ public abstract class ScreenImpl extends Screen implements ScreenModeListener {
             ScreenModeStatus.unlockScreenModeStatus();
         }
     }
+    
+    private final void shutdown() {
+        ScreenModeStatus sms = ScreenModeStatus.getScreenModeStatusUnlocked(getFQName());
+        if(null != sms) {
+            if(!sms.isOriginalMode()) {
+                try {
+                    setCurrentScreenModeImpl(sms.getOriginalScreenMode());
+                } catch (Throwable t) {
+                    // be quiet .. shutdown
+                }
+            }
+            ScreenModeStatus.unmapScreenModeStatusUnlocked(getFQName());
+        }            
+    }
+    private static final void shutdownAll() {
+        for(int i=0; i < screenList.size(); i++) {
+            ((ScreenImpl)screenList.get(i)).shutdown();
+        }
+    }
+    
+    private static synchronized void registerShutdownHook() {
+        final Thread shutdownHook = new Thread(new Runnable() {
+            public void run() {
+                ScreenImpl.shutdownAll();
+            }
+        });
+        AccessController.doPrivileged(new PrivilegedAction() {
+            public Object run() {
+                Runtime.getRuntime().addShutdownHook(shutdownHook);
+                return null;
+            }
+        });
+    }
 }
 
diff --git a/src/newt/classes/jogamp/newt/ScreenModeStatus.java b/src/newt/classes/jogamp/newt/ScreenModeStatus.java
index 4d8b8b5f6..dfd4e99a6 100644
--- a/src/newt/classes/jogamp/newt/ScreenModeStatus.java
+++ b/src/newt/classes/jogamp/newt/ScreenModeStatus.java
@@ -34,6 +34,7 @@ import com.jogamp.common.util.locks.RecursiveLock;
 import com.jogamp.newt.Screen;
 import com.jogamp.newt.ScreenMode;
 import com.jogamp.newt.event.ScreenModeListener;
+
 import java.util.ArrayList;
 import java.util.HashMap;
 
@@ -41,19 +42,19 @@ public class ScreenModeStatus {
     private static boolean DEBUG = Screen.DEBUG;
 
     private RecursiveLock lock = new RecursiveLock();
-    private ArrayHashSet/*<ScreenMode>*/ screenModes;
+    private ArrayHashSet<ScreenMode> screenModes;
     private IntIntHashMap screenModesIdx2NativeIdx;
     private ScreenMode currentScreenMode;
     private ScreenMode originalScreenMode;
-    private ArrayList/*<ScreenModeChangeListener>*/ listener = new ArrayList();
+    private ArrayList<ScreenModeListener> listener = new ArrayList<ScreenModeListener>();
 
-    private static HashMap screenFQN2ScreenModeStatus = new HashMap();
+    private static HashMap<String, ScreenModeStatus> screenFQN2ScreenModeStatus = new HashMap<String, ScreenModeStatus>();
     private static RecursiveLock screen2ScreenModeStatusLock = new RecursiveLock();
 
     protected static void mapScreenModeStatus(String screenFQN, ScreenModeStatus sms) {
         screen2ScreenModeStatusLock.lock();
         try {
-            ScreenModeStatus _sms = (ScreenModeStatus) screenFQN2ScreenModeStatus.get(screenFQN);
+            ScreenModeStatus _sms = screenFQN2ScreenModeStatus.get(screenFQN);
             if( null != _sms ) {
                 throw new RuntimeException("ScreenModeStatus "+_sms+" already mapped to "+screenFQN);
             }
@@ -73,23 +74,29 @@ public class ScreenModeStatus {
     protected static void unmapScreenModeStatus(String screenFQN) {
         screen2ScreenModeStatusLock.lock();
         try {
-            ScreenModeStatus sms = (ScreenModeStatus) screenFQN2ScreenModeStatus.remove(screenFQN);
-            if(DEBUG) {
-                System.err.println("ScreenModeStatus.unmap "+screenFQN+" -> "+sms);
-            }
+            unmapScreenModeStatusUnlocked(screenFQN);
         } finally {
             screen2ScreenModeStatusLock.unlock();
         }
     }
+    protected static void unmapScreenModeStatusUnlocked(String screenFQN) {
+        ScreenModeStatus sms = screenFQN2ScreenModeStatus.remove(screenFQN);
+        if(DEBUG) {
+            System.err.println("ScreenModeStatus.unmap "+screenFQN+" -> "+sms);
+        }
+    }
 
     protected static ScreenModeStatus getScreenModeStatus(String screenFQN) {
         screen2ScreenModeStatusLock.lock();
         try {
-            return (ScreenModeStatus) screenFQN2ScreenModeStatus.get(screenFQN);
+            return getScreenModeStatusUnlocked(screenFQN);
         } finally {
             screen2ScreenModeStatusLock.unlock();
         }
     }
+    protected static ScreenModeStatus getScreenModeStatusUnlocked(String screenFQN) {
+        return screenFQN2ScreenModeStatus.get(screenFQN);
+    }
 
     protected static void lockScreenModeStatus() {
         screen2ScreenModeStatusLock.lock();
@@ -99,7 +106,7 @@ public class ScreenModeStatus {
         screen2ScreenModeStatusLock.unlock();
     }
     
-    public ScreenModeStatus(ArrayHashSet/*<ScreenMode>*/ screenModes,
+    public ScreenModeStatus(ArrayHashSet<ScreenMode> screenModes,
                             IntIntHashMap screenModesIdx2NativeIdx) {
         this.screenModes = screenModes;
         this.screenModesIdx2NativeIdx = screenModesIdx2NativeIdx;
@@ -135,7 +142,7 @@ public class ScreenModeStatus {
         }
     }
 
-    protected final ArrayHashSet/*<ScreenMode>*/ getScreenModes() {
+    protected final ArrayHashSet<ScreenMode> getScreenModes() {
         return screenModes;
     }
 
@@ -175,7 +182,7 @@ public class ScreenModeStatus {
         lock();
         try {
             for(int i=0; i<listener.size(); i++) {
-                ((ScreenModeListener)listener.get(i)).screenModeChangeNotify(desiredScreenMode);
+                listener.get(i).screenModeChangeNotify(desiredScreenMode);
             }
         } finally {
             unlock();
@@ -189,7 +196,7 @@ public class ScreenModeStatus {
                 this.currentScreenMode = currentScreenMode;
             }
             for(int i=0; i<listener.size(); i++) {
-                ((ScreenModeListener)listener.get(i)).screenModeChanged(currentScreenMode, success);
+                listener.get(i).screenModeChanged(currentScreenMode, success);
             }
         } finally {
             unlock();
@@ -203,5 +210,4 @@ public class ScreenModeStatus {
     protected final void unlock() throws RuntimeException {
         lock.unlock();
     }
-
 }
diff --git a/src/test/com/jogamp/opengl/test/junit/newt/ManualScreenMode03NEWT.java b/src/test/com/jogamp/opengl/test/junit/newt/ManualScreenMode03NEWT.java
new file mode 100644
index 000000000..29ec443f7
--- /dev/null
+++ b/src/test/com/jogamp/opengl/test/junit/newt/ManualScreenMode03NEWT.java
@@ -0,0 +1,110 @@
+/**
+ * Copyright 2011 JogAmp Community. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without modification, are
+ * permitted provided that the following conditions are met:
+ * 
+ *    1. Redistributions of source code must retain the above copyright notice, this list of
+ *       conditions and the following disclaimer.
+ * 
+ *    2. Redistributions in binary form must reproduce the above copyright notice, this list
+ *       of conditions and the following disclaimer in the documentation and/or other materials
+ *       provided with the distribution.
+ * 
+ * THIS SOFTWARE IS PROVIDED BY JogAmp Community ``AS IS'' AND ANY EXPRESS OR IMPLIED
+ * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
+ * FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL JogAmp Community OR
+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
+ * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * 
+ * The views and conclusions contained in the software and documentation are those of the
+ * authors and should not be interpreted as representing official policies, either expressed
+ * or implied, of JogAmp Community.
+ */
+ 
+package com.jogamp.opengl.test.junit.newt;
+
+import java.io.IOException;
+import javax.media.opengl.GLCapabilities;
+import javax.media.opengl.GLProfile;
+
+import com.jogamp.opengl.util.Animator;
+
+import com.jogamp.newt.Display;
+import com.jogamp.newt.NewtFactory;
+import com.jogamp.newt.Screen;
+import com.jogamp.newt.ScreenMode;
+import com.jogamp.newt.opengl.GLWindow;
+import com.jogamp.newt.util.ScreenModeUtil;
+import com.jogamp.opengl.test.junit.jogl.demos.es2.GearsES2;
+import com.jogamp.opengl.test.junit.util.UITestCase;
+import java.util.List;
+import javax.media.nativewindow.util.Dimension;
+
+/**
+ * Manual testing the ScreenImpl shutdown hook, 
+ * which shall reset the ScreenMode to it's original state
+ * when the application exists (normal or ctrl-c).
+ */
+public class ManualScreenMode03NEWT extends UITestCase {
+    static int waitTime = 7000; // 1 sec
+    
+    static GLWindow createWindow(Screen screen, GLCapabilities caps, int width, int height, boolean onscreen, boolean undecorated) {
+        caps.setOnscreen(onscreen);
+
+        GLWindow window = GLWindow.create(screen, caps);
+        window.setSize(width, height);
+        window.addGLEventListener(new GearsES2());
+        window.setVisible(true);
+        return window;
+    }
+
+    public void run() {
+        int width  = 640;
+        int height = 480;
+        GLProfile glp = GLProfile.getDefault();
+        GLCapabilities caps = new GLCapabilities(glp);
+        Display display = NewtFactory.createDisplay(null); // local display
+        Screen screen  = NewtFactory.createScreen(display, 0); // screen 0
+        GLWindow window = createWindow(screen, caps, width, height, true /* onscreen */, false /* undecorated */);
+
+        List<ScreenMode> screenModes = screen.getScreenModes();
+        if(null==screenModes) {
+            // no support ..
+            System.err.println("Your platform has no ScreenMode change support, sorry");
+            return;
+        }
+        Animator animator = new Animator(window);
+        animator.start();
+
+        ScreenMode smCurrent = screen.getCurrentScreenMode();
+        ScreenMode smOrig = screen.getOriginalScreenMode();
+        System.err.println("[0] current/orig: "+smCurrent);
+
+        screenModes = ScreenModeUtil.filterByRate(screenModes, smOrig.getMonitorMode().getRefreshRate());
+        screenModes = ScreenModeUtil.filterByRotation(screenModes, 0);
+        screenModes = ScreenModeUtil.filterByResolution(screenModes, new Dimension(801, 601));
+        screenModes = ScreenModeUtil.getHighestAvailableBpp(screenModes);
+
+        ScreenMode sm = (ScreenMode) screenModes.get(0);
+        System.err.println("[0] set current: "+sm);
+        screen.setCurrentScreenMode(sm);
+
+        System.err.print("[0] post setting .. wait <");
+        try {
+            Thread.sleep(waitTime);
+        } catch (InterruptedException e) {
+        }
+        System.err.println("done>");
+        System.exit(0);
+    }
+
+    public static void main(String args[]) throws IOException {
+        ManualScreenMode03NEWT t = new ManualScreenMode03NEWT();
+        t.run();
+    }
+}
diff --git a/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode01NEWT.java b/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode01NEWT.java
index 60384b1c2..c84db6081 100644
--- a/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode01NEWT.java
+++ b/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode01NEWT.java
@@ -46,7 +46,7 @@ import com.jogamp.newt.Window;
 import com.jogamp.newt.ScreenMode;
 import com.jogamp.newt.opengl.GLWindow;
 import com.jogamp.newt.util.ScreenModeUtil;
-import com.jogamp.opengl.test.junit.jogl.demos.gl2.Gears;
+import com.jogamp.opengl.test.junit.jogl.demos.es2.GearsES2;
 import com.jogamp.opengl.test.junit.util.UITestCase;
 import java.util.List;
 import javax.media.nativewindow.util.Dimension;
@@ -55,8 +55,8 @@ public class TestScreenMode01NEWT extends UITestCase {
     static GLProfile glp;
     static int width, height;
     
-    static int waitTimeShort = 1000; // 1 sec
-    static int waitTimeLong = 5000; // 5 sec
+    static int waitTimeShort = 2000; // 2 sec
+    static int waitTimeLong = 8000; // 8 sec
     
     
 
@@ -74,7 +74,7 @@ public class TestScreenMode01NEWT extends UITestCase {
 
         GLWindow window = GLWindow.create(screen, caps);
         window.setSize(width, height);
-        window.addGLEventListener(new Gears());
+        window.addGLEventListener(new GearsES2());
         Assert.assertNotNull(window);
         window.setVisible(true);
         return window;
@@ -126,7 +126,7 @@ public class TestScreenMode01NEWT extends UITestCase {
         GLWindow window = createWindow(screen, caps, width, height, true /* onscreen */, false /* undecorated */);
         Assert.assertNotNull(window);
 
-        List screenModes = screen.getScreenModes();
+        List<ScreenMode> screenModes = screen.getScreenModes();
         if(null==screenModes) {
             // no support ..
             System.err.println("Your platform has no ScreenMode change support, sorry");
@@ -225,7 +225,7 @@ public class TestScreenMode01NEWT extends UITestCase {
         animator.start();
 
         ScreenMode smOrig = screen.getOriginalScreenMode();
-        List screenModes = screen.getScreenModes();
+        List<ScreenMode> screenModes = screen.getScreenModes();
         if(null==screenModes) {
             // no support ..
             destroyWindow(window);
diff --git a/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode02NEWT.java b/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode02NEWT.java
index 26b64945f..a9157a9ee 100644
--- a/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode02NEWT.java
+++ b/src/test/com/jogamp/opengl/test/junit/newt/TestScreenMode02NEWT.java
@@ -46,7 +46,7 @@ import com.jogamp.newt.Window;
 import com.jogamp.newt.ScreenMode;
 import com.jogamp.newt.opengl.GLWindow;
 import com.jogamp.newt.util.ScreenModeUtil;
-import com.jogamp.opengl.test.junit.jogl.demos.gl2.Gears;
+import com.jogamp.opengl.test.junit.jogl.demos.es2.GearsES2;
 import com.jogamp.opengl.test.junit.util.UITestCase;
 import java.util.List;
 import javax.media.nativewindow.util.Dimension;
@@ -55,8 +55,8 @@ public class TestScreenMode02NEWT extends UITestCase {
     static GLProfile glp;
     static int width, height;
     
-    static int waitTimeShort = 1000; // 1 sec
-    static int waitTimeLong = 5000; // 5 sec
+    static int waitTimeShort = 2000; // 2 sec
+    static int waitTimeLong = 8000; // 8 sec
 
     @BeforeClass
     public static void initClass() {
@@ -72,7 +72,7 @@ public class TestScreenMode02NEWT extends UITestCase {
 
         GLWindow window = GLWindow.create(screen, caps);
         window.setSize(width, height);
-        window.addGLEventListener(new Gears());
+        window.addGLEventListener(new GearsES2());
         Assert.assertNotNull(window);
         window.setVisible(true);
         Assert.assertTrue(window.isVisible());
@@ -98,7 +98,7 @@ public class TestScreenMode02NEWT extends UITestCase {
         GLWindow window = createWindow(screen, caps, width, height, true /* onscreen */, false /* undecorated */);
         Assert.assertNotNull(window);
 
-        List screenModes = screen.getScreenModes();
+        List<ScreenMode> screenModes = screen.getScreenModes();
         if(null==screenModes) {
             // no support ..
             System.err.println("Your platform has no ScreenMode change support, sorry");