commit f95aae25a6ba450b534a196f86c91c82b48a2d17
author George Mount <mount@google.com> Sun Jan 13 14:39:47 2013 -0800
committer Android (Google) Code Review <android-gerrit@google.com> Sun Jan 13 14:39:48 2013 -0800
tree c7451c0209758fc2fa952f28a49103a37ab60af2
parent 8e3b1df4fcc770965d0796ac2046ef9e265d4bdb
parent 6f27f26e83d53640675f6b9606d01acd936dd5a2

Merge "Fix LightCycle capture NullPointerException." into gb-ub-photos-bryce

src/com/android/camera/CameraScreenNail.java

diff --git a/src/com/android/camera/CameraScreenNail.java b/src/com/android/camera/CameraScreenNail.java
index 14a7ac4..d2c62fd 100644
--- a/src/com/android/camera/CameraScreenNail.java
+++ b/src/com/android/camera/CameraScreenNail.java
@@ -85,16 +85,11 @@
         }
 
         @Override
-        public void onInitialize(GLCanvas canvas) {
-        }
-
-        @Override
         public boolean requiresSurfaceTexture() {
             return true;
         }
     };
     private DrawClient mDraw = mDefaultDraw;
-    private boolean mNeedsInitialize = false;
 
     public interface Listener {
         void requestRender();
@@ -111,8 +106,6 @@
     public interface DrawClient {
         void onDraw(GLCanvas canvas, int x, int y, int width, int height);
 
-        void onInitialize(GLCanvas canvas);
-
         boolean requiresSurfaceTexture();
     }
 
@@ -271,6 +264,10 @@
         }
     }
 
+    public RawTexture getAnimationTexture() {
+        return mAnimTexture;
+    }
+
     public void animateFlash(int displayRotation) {
         synchronized (mLock) {
             mCaptureAnimManager.setOrientation(displayRotation);
@@ -326,7 +323,6 @@
             } else {
                 mDraw = draw;
             }
-            mNeedsInitialize = true;
         }
         mListener.requestRender();
     }
@@ -334,10 +330,6 @@
     @Override
     public void draw(GLCanvas canvas, int x, int y, int width, int height) {
         synchronized (mLock) {
-            if (mNeedsInitialize) {
-                mDraw.onInitialize(canvas);
-                mNeedsInitialize = false;
-            }
             allocateTextureIfRequested(canvas);
             if (!mVisible) mVisible = true;
             SurfaceTexture surfaceTexture = getSurfaceTexture();
@@ -396,7 +388,7 @@
                     // Continue to the normal draw procedure if the animation is
                     // not drawn.
                     mAnimState = ANIM_NONE;
-                    super.draw(canvas, x, y, width, height);
+                    directDraw(canvas, x, y, width, height);
                 }
             }
             callbackIfNeeded();
@@ -404,17 +396,24 @@
     }
 
     private void copyPreviewTexture(GLCanvas canvas) {
+        if (!mDraw.requiresSurfaceTexture() && mAnimTexture == null) {
+            mAnimTexture = new RawTexture(getTextureWidth(), getTextureHeight(), true);
+            mAnimTexture.setIsFlippedVertically(true);
+        }
         int width = mAnimTexture.getWidth();
         int height = mAnimTexture.getHeight();
         canvas.beginRenderTarget(mAnimTexture);
-        // Flip preview texture vertically. OpenGL uses bottom left point
-        // as the origin (0, 0).
-        canvas.translate(0, height);
-        canvas.scale(1, -1, 1);
-        getSurfaceTexture().getTransformMatrix(mTextureTransformMatrix);
-        updateTransformMatrix(mTextureTransformMatrix);
-        canvas.drawTexture(mExtTexture,
-                mTextureTransformMatrix, 0, 0, width, height);
+        if (!mDraw.requiresSurfaceTexture()) {
+            mDraw.onDraw(canvas, 0, 0, width, height);
+        } else {
+            // Flip preview texture vertically. OpenGL uses bottom left point
+            // as the origin (0, 0).
+            canvas.translate(0, height);
+            canvas.scale(1, -1, 1);
+            getSurfaceTexture().getTransformMatrix(mTextureTransformMatrix);
+            updateTransformMatrix(mTextureTransformMatrix);
+            canvas.drawTexture(mExtTexture, mTextureTransformMatrix, 0, 0, width, height);
+        }
         canvas.endRenderTarget();
     }