commit c4e9a56ed868a4fd0b012b85aded3d5d9b783d86
author Tracy Zhou <tracyzhou@google.com> Mon Jun 22 01:12:23 2020 -0700
committer Tracy Zhou <tracyzhou@google.com> Mon Jun 22 09:33:52 2020 -0700
tree 0997dd038ab532ccd13a6edb899af439fe48f083
parent 07bfdd5bd4d05563ea8702a5a82ff729389cb6bf

No need to remove callback and call surface.release in GridOptionPreviewer

It's taken care of by SurfaceControlViewHost.

Bug: 155365052
Test: manual
Change-Id: Ibc25857cfca489cf20d5b80eb68d8618b312c140

src/com/android/customization/picker/grid/GridOptionPreviewer.java

diff --git a/src/com/android/customization/picker/grid/GridOptionPreviewer.java b/src/com/android/customization/picker/grid/GridOptionPreviewer.java
index 47303e9..fdefa05 100644
--- a/src/com/android/customization/picker/grid/GridOptionPreviewer.java
+++ b/src/com/android/customization/picker/grid/GridOptionPreviewer.java
@@ -17,7 +17,6 @@
 
 import android.content.Context;
 import android.os.Bundle;
-import android.view.Surface;
 import android.view.SurfaceHolder;
 import android.view.SurfaceView;
 import android.view.ViewGroup;
@@ -54,11 +53,6 @@
     public void release() {
         if (mGridOptionSurface != null) {
             mSurfaceCallback.cleanUp();
-            mGridOptionSurface.getHolder().removeCallback(mSurfaceCallback);
-            Surface surface = mGridOptionSurface.getHolder().getSurface();
-            if (surface != null) {
-                surface.release();
-            }
             mGridOptionSurface = null;
         }
         mPreviewContainer.removeAllViews();