Add more temporary logging for investigating detachFunctor

bug:6596807
Change-Id: Ic9e34e323b12a887f2e8df0773a6155627b6a64f

Author: ChrisCraik

core/java/android/view/HardwareRenderer.java

@@ -1229,7 +1229,7 @@ private void scheduleFunctors(View.AttachInfo attachInfo) {
         void detachFunctor(int functor) {
             if (mCanvas != null) {
                 mCanvas.detachFunctor(functor);
-            }
+            } else Log.e(LOG_TAG, "can't detachFunctor, have no canvas");
         }
 
         @Override

core/java/android/view/ViewRootImpl.java

@@ -678,6 +678,9 @@ public boolean attachFunctor(int functor) {
     }
 
     public void detachFunctor(int functor) {
+        Log.e(HardwareRenderer.LOG_TAG, String.format("detachFunctor, mHarwareRenderer==null %b, enabled %b",
+                mAttachInfo.mHardwareRenderer == null,
+                mAttachInfo.mHardwareRenderer == null ? false : mAttachInfo.mHardwareRenderer.isEnabled()));
         if (mAttachInfo.mHardwareRenderer != null && mAttachInfo.mHardwareRenderer.isEnabled()) {
             mAttachInfo.mHardwareRenderer.detachFunctor(functor);
         }

libs/hwui/OpenGLRenderer.cpp

@@ -262,7 +262,10 @@ void OpenGLRenderer::resume() {
 }
 
 void OpenGLRenderer::detachFunctor(Functor* functor) {
-    mFunctors.remove(functor);
+    int size = mFunctors.size();
+    int remove = mFunctors.remove(functor);
+    ALOGD("OGLR %p detachFunctor %p, removed at index %d of %d",
+          this, functor, remove, size);
 }
 
 void OpenGLRenderer::attachFunctor(Functor* functor) {
@@ -309,7 +312,7 @@ status_t OpenGLRenderer::invokeFunctors(Rect& dirty) {
 
 status_t OpenGLRenderer::callDrawGLFunction(Functor* functor, Rect& dirty) {
     interrupt();
-    detachFunctor(functor);
+    mFunctors.remove(functor);
 
     if (mDirtyClip) {
         setScissorFromClip();