Bug 1207 - GLDebugMessageHandler: Support GL_KHR_debug for Desktop and ES profile

GL_KHR_debug <https://www.opengl.org/registry/specs/KHR/debug.txt>

GL_KHR_debug shall be favorized before
  - GL_ARB_debug_output
  - GL_AMD_debug_output

Allow GL_KHR_debug for GL2GL3 and GL2ES2 profiles,
i.e. including ES profiles: GLES2, GLES3.

GL_ARB_debug_output and GL_AMD_debug_output
are only allowed for desktop GL2GL3 profiles.

2 files changed, 66 insertions, 30 deletions

diff --git a/src/jogl/classes/jogamp/opengl/GLContextImpl.java b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
index 45cfc6bcc..6e904d0ed 100644
--- a/src/jogl/classes/jogamp/opengl/GLContextImpl.java
+++ b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
@@ -693,7 +693,7 @@ public abstract class GLContextImpl extends GLContext {
         // throws an GLException if not
         // FIXME: drawable.getGLProfile().verifyEquality(gl.getGLProfile());
 
-        glDebugHandler.init( isGL2GL3() && isGLDebugEnabled() );
+        glDebugHandler.init( isGLDebugEnabled() );
 
         if(DEBUG_GL) {
             setGL( GLPipelineFactory.create("com.jogamp.opengl.Debug", null, gl, null) );
@@ -2820,8 +2820,8 @@ public abstract class GLContextImpl extends GLContext {
 
   @Override
   public final void glDebugMessageControl(final int source, final int type, final int severity, final int count, final IntBuffer ids, final boolean enabled) {
-      if(glDebugHandler.isExtensionARB()) {
-          gl.getGL2GL3().glDebugMessageControl(source, type, severity, count, ids, enabled);
+      if(glDebugHandler.isExtensionKHRARB()) {
+          gl.getGL2ES2().glDebugMessageControl(source, type, severity, count, ids, enabled);
       } else if(glDebugHandler.isExtensionAMD()) {
           gl.getGL2GL3().glDebugMessageEnableAMD(GLDebugMessage.translateARB2AMDCategory(source, type), severity, count, ids, enabled);
       }
@@ -2829,8 +2829,8 @@ public abstract class GLContextImpl extends GLContext {
 
   @Override
   public final void glDebugMessageControl(final int source, final int type, final int severity, final int count, final int[] ids, final int ids_offset, final boolean enabled) {
-      if(glDebugHandler.isExtensionARB()) {
-          gl.getGL2GL3().glDebugMessageControl(source, type, severity, count, ids, ids_offset, enabled);
+      if(glDebugHandler.isExtensionKHRARB()) {
+          gl.getGL2ES2().glDebugMessageControl(source, type, severity, count, ids, ids_offset, enabled);
       } else if(glDebugHandler.isExtensionAMD()) {
           gl.getGL2GL3().glDebugMessageEnableAMD(GLDebugMessage.translateARB2AMDCategory(source, type), severity, count, ids, ids_offset, enabled);
       }
@@ -2839,8 +2839,8 @@ public abstract class GLContextImpl extends GLContext {
   @Override
   public final void glDebugMessageInsert(final int source, final int type, final int id, final int severity, final String buf) {
       final int len = (null != buf) ? buf.length() : 0;
-      if(glDebugHandler.isExtensionARB()) {
-          gl.getGL2GL3().glDebugMessageInsert(source, type, id, severity, len, buf);
+      if(glDebugHandler.isExtensionKHRARB()) {
+          gl.getGL2ES2().glDebugMessageInsert(source, type, id, severity, len, buf);
       } else if(glDebugHandler.isExtensionAMD()) {
           gl.getGL2GL3().glDebugMessageInsertAMD(GLDebugMessage.translateARB2AMDCategory(source, type), severity, id, len, buf);
       }
diff --git a/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java b/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java
index cd31d02fd..fb4529da4 100644
--- a/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java
+++ b/src/jogl/classes/jogamp/opengl/GLDebugMessageHandler.java
@@ -61,8 +61,9 @@ import com.jogamp.opengl.GLExtensions;
 public class GLDebugMessageHandler {
     private static final boolean DEBUG = Debug.debug("GLDebugMessageHandler");
 
-    private static final int EXT_ARB = 1;
-    private static final int EXT_AMD = 2;
+    private static final int EXT_KHR = 1;
+    private static final int EXT_ARB = 2;
+    private static final int EXT_AMD = 3;
 
     static {
         if ( !initIDs0() ) {
@@ -75,6 +76,7 @@ public class GLDebugMessageHandler {
 
     // licefycle: init - EOL
     private String extName;
+    private String extSuffix;
     private int extType;
     private long glDebugMessageCallbackProcAddress;
     private boolean extAvailable;
@@ -92,6 +94,7 @@ public class GLDebugMessageHandler {
         this.listenerImpl = new ListenerSyncedImplStub<GLDebugListener>();
         this.glDebugMessageCallbackProcAddress = 0;
         this.extName = null;
+        this.extSuffix = null;
         this.extType = 0;
         this.extAvailable = false;
         this.handle = 0;
@@ -143,43 +146,68 @@ public class GLDebugMessageHandler {
             }
             return;
         }
-        if( ctx.isExtensionAvailable(GLExtensions.ARB_debug_output) ) {
+        if( ctx.isExtensionAvailable(GLExtensions.GL_KHR_debug) ) {
+            extName = GLExtensions.GL_KHR_debug;
+            extSuffix = ctx.isGLES() ? "KHR" : ""; // See SPEC!
+            extType = EXT_KHR;
+        } else if( ctx.isExtensionAvailable(GLExtensions.ARB_debug_output) ) {
             extName = GLExtensions.ARB_debug_output;
+            extSuffix = "ARB";
             extType = EXT_ARB;
         } else if( ctx.isExtensionAvailable(GLExtensions.AMD_debug_output) ) {
             extName = GLExtensions.AMD_debug_output;
+            extSuffix = "AMD";
             extType = EXT_AMD;
         }
-        if(DEBUG) {
-            System.err.println("GLDebugMessageHandler: Using extension: <"+extName+">");
+
+        // Validate GL Profile, just to be sure
+        switch(extType) {
+            case EXT_KHR:
+                if( !ctx.isGL2ES2() ) {
+                    if(DEBUG) {
+                        System.err.println("Non GL2ES2 context not supported, has "+ctx.getGLVersion());
+                    }
+                    extType = 0;
+                }
+                break;
+            case EXT_ARB:
+                // fall through intended
+            case EXT_AMD:
+                if( !ctx.isGL2GL3() ) {
+                    if(DEBUG) {
+                        System.err.println("Non GL2GL3 context not supported, has "+ctx.getGLVersion());
+                    }
+                    extType = 0;
+                }
+                break;
         }
 
         if(0 == extType) {
+            extName = null;
+            extSuffix = null;
             if(DEBUG) {
                 System.err.println("GLDebugMessageHandler: No extension available! "+ctx.getGLVersion());
                 System.err.println("GL_EXTENSIONS  "+ctx.getGLExtensionCount());
                 System.err.println(ctx.getGLExtensionsString());
             }
             return;
+        } else  if(DEBUG) {
+            System.err.println("GLDebugMessageHandler: Using extension: <"+extName+"> with suffix <"+extSuffix+">");
         }
 
         final ProcAddressTable procAddressTable = ctx.getGLProcAddressTable();
-        if( !ctx.isGLES1() && !ctx.isGLES2() ) {
-            switch(extType) {
-                case EXT_ARB:
-                    glDebugMessageCallbackProcAddress = getAddressFor(procAddressTable, "glDebugMessageCallbackARB");
-                    break;
-                case EXT_AMD:
-                    glDebugMessageCallbackProcAddress = getAddressFor(procAddressTable, "glDebugMessageCallbackAMD");
-                    break;
-            }
-        } else {
-            glDebugMessageCallbackProcAddress = 0;
-            if(DEBUG) {
-                System.err.println("Non desktop context not supported");
-            }
+        switch(extType) {
+            case EXT_KHR:
+                glDebugMessageCallbackProcAddress = getAddressFor(procAddressTable, "glDebugMessageCallback"+extSuffix);
+                break;
+            case EXT_ARB:
+                glDebugMessageCallbackProcAddress = getAddressFor(procAddressTable, "glDebugMessageCallback"+extSuffix);
+                break;
+            case EXT_AMD:
+                glDebugMessageCallbackProcAddress = getAddressFor(procAddressTable, "glDebugMessageCallback"+extSuffix);
+                break;
         }
-        extAvailable = 0 < extType && null != extName && 0 != glDebugMessageCallbackProcAddress;
+        extAvailable = 0 < extType && null != extName && null != extSuffix && 0 != glDebugMessageCallbackProcAddress;
 
         if(DEBUG) {
             System.err.println("GLDebugMessageHandler: extAvailable: "+extAvailable+", glDebugMessageCallback* : 0x"+Long.toHexString(glDebugMessageCallbackProcAddress));
@@ -203,12 +231,20 @@ public class GLDebugMessageHandler {
         return extName;
     }
 
+    public final boolean isExtensionKHRARB() {
+        return EXT_KHR == extType || EXT_ARB == extType;
+    }
+
+    public final boolean isExtensionKHR() {
+        return EXT_KHR == extType;
+    }
+
     public final boolean isExtensionARB() {
-        return extName == GLExtensions.ARB_debug_output;
+        return EXT_ARB == extType;
     }
 
     public final boolean isExtensionAMD() {
-        return extName == GLExtensions.AMD_debug_output;
+        return EXT_AMD == extType;
     }
 
     /**
@@ -226,7 +262,7 @@ public class GLDebugMessageHandler {
         }
     }
     private final void setSynchronousImpl() {
-        if(isExtensionARB()) {
+        if(isExtensionKHRARB()) {
             if(synchronous) {
                 ctx.getGL().glEnable(GL2ES2.GL_DEBUG_OUTPUT_SYNCHRONOUS);
             } else {