Use custom GLSurfaceView

Ensure GLRootView's mContentView.detachFromRoot() called in render thread

app/src/main/java/com/hippo/gl/view/EGLError.java

@@ -0,0 +1,65 @@
+/*
+ * Copyright (C) 2008 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.hippo.gl.view;
+
+import javax.microedition.khronos.egl.EGL10;
+import javax.microedition.khronos.egl.EGL11;
+
+final class EGLError {
+    private EGLError() {}
+
+    private static String getHex(int value) {
+        return "0x" + Integer.toHexString(value);
+    }
+
+    public static String getErrorString(int error) {
+        switch (error) {
+        case EGL10.EGL_SUCCESS:
+            return "EGL_SUCCESS";
+        case EGL10.EGL_NOT_INITIALIZED:
+            return "EGL_NOT_INITIALIZED";
+        case EGL10.EGL_BAD_ACCESS:
+            return "EGL_BAD_ACCESS";
+        case EGL10.EGL_BAD_ALLOC:
+            return "EGL_BAD_ALLOC";
+        case EGL10.EGL_BAD_ATTRIBUTE:
+            return "EGL_BAD_ATTRIBUTE";
+        case EGL10.EGL_BAD_CONFIG:
+            return "EGL_BAD_CONFIG";
+        case EGL10.EGL_BAD_CONTEXT:
+            return "EGL_BAD_CONTEXT";
+        case EGL10.EGL_BAD_CURRENT_SURFACE:
+            return "EGL_BAD_CURRENT_SURFACE";
+        case EGL10.EGL_BAD_DISPLAY:
+            return "EGL_BAD_DISPLAY";
+        case EGL10.EGL_BAD_MATCH:
+            return "EGL_BAD_MATCH";
+        case EGL10.EGL_BAD_NATIVE_PIXMAP:
+            return "EGL_BAD_NATIVE_PIXMAP";
+        case EGL10.EGL_BAD_NATIVE_WINDOW:
+            return "EGL_BAD_NATIVE_WINDOW";
+        case EGL10.EGL_BAD_PARAMETER:
+            return "EGL_BAD_PARAMETER";
+        case EGL10.EGL_BAD_SURFACE:
+            return "EGL_BAD_SURFACE";
+        case EGL11.EGL_CONTEXT_LOST:
+            return "EGL_CONTEXT_LOST";
+        default:
+            return getHex(error);
+        }
+    }
+}

app/src/main/java/com/hippo/gl/view/GLRootView.java

@@ -20,7 +20,6 @@ import android.annotation.TargetApi;
 import android.content.Context;
 import android.graphics.Matrix;
 import android.graphics.PixelFormat;
-import android.opengl.GLSurfaceView;
 import android.os.Build;
 import android.os.Parcelable;
 import android.os.Process;
@@ -53,6 +52,7 @@ import javax.microedition.khronos.egl.EGLDisplay;
 import javax.microedition.khronos.opengles.GL10;
 import javax.microedition.khronos.opengles.GL11;
 
+// TODO Call attachToRoot and detachFromRoot in render thread
 // The root component of all <code>GLView</code>s. The rendering is done in GL
 // thread while the event handling is done in the main thread.  To synchronize
 // the two threads, the entry points of this package need to synchronize on the
@@ -315,6 +315,13 @@ public class GLRootView extends GLSurfaceView
         mCanvas.setSize(width, height);
     }
 
+    @Override
+    public void onSurfaceDestroyed() {
+        if (mContentView != null && mContentView.isAttachedToRoot()) {
+            mContentView.detachFromRoot();
+        }
+    }
+
     private void outputFps() {
         long now = System.nanoTime();
         if (mFrameCountingStart == 0) {
@@ -573,28 +580,18 @@ public class GLRootView extends GLSurfaceView
     public void surfaceChanged(SurfaceHolder holder, int format, int w, int h) {
         unfreeze();
         super.surfaceChanged(holder, format, w, h);
-
-        requestLayoutContentPane();
     }
 
     @Override
     public void surfaceCreated(SurfaceHolder holder) {
         unfreeze();
         super.surfaceCreated(holder);
-
-        if (mContentView != null && !mContentView.isAttachedToRoot()) {
-            mContentView.attachToRoot(this);
-        }
     }
 
     @Override
     public void surfaceDestroyed(SurfaceHolder holder) {
         unfreeze();
         super.surfaceDestroyed(holder);
-
-        if (mContentView != null && mContentView.isAttachedToRoot()) {
-            mContentView.detachFromRoot();
-        }
     }
 
     @Override