fix vignette startup issues

Change-Id: I3a8461edf3239044565611ed35b41cb87e5a529f

diff --git a/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java b/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java
index 2479188..b4ca8e1 100644 (file)
--- a/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java
+++ b/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java
@@ -249,4 +249,8 @@ public class EclipseControl {
         fillHandles(canvas, mCenterX, mCenterY, mRadiusX, mRadiusY);
         paintPoint(canvas, mCenterX, mCenterY);
     }
+
+    public boolean isUndefined() {
+        return Float.isNaN(mCenterX);
+    }
 }

diff --git a/src/com/android/gallery3d/filtershow/imageshow/ImageVignette.java b/src/com/android/gallery3d/filtershow/imageshow/ImageVignette.java
index d56cc47..729ac7f 100644 (file)
--- a/src/com/android/gallery3d/filtershow/imageshow/ImageVignette.java
+++ b/src/com/android/gallery3d/filtershow/imageshow/ImageVignette.java
@@ -92,17 +92,18 @@ public class ImageVignette extends ImageShow {
         if (Float.isNaN(c[0])) {
             float cx = mImageLoader.getOriginalBounds().width() / 2;
             float cy = mImageLoader.getOriginalBounds().height() / 2;
-            float rx = cx * .8f;
-            float ry = cy * .8f;
+            float rx = Math.min(cx, cy) * .8f;
+            float ry = rx;
             mVignetteRep.setCenter(cx, cy);
             mVignetteRep.setRadius(rx, ry);
 
             c[0] = cx;
             c[1] = cy;
             toScr.mapPoints(c);
-
-            mElipse.setCenter(c[0], c[1]);
-            mElipse.setRadius(c[0] * 0.8f, c[1] * 0.8f);
+            if (getWidth() != 0) {
+                mElipse.setCenter(c[0], c[1]);
+                mElipse.setRadius(c[0] * 0.8f, c[1] * 0.8f);
+            }
         } else {
 
             toScr.mapPoints(c);
@@ -120,6 +121,9 @@ public class ImageVignette extends ImageShow {
     @Override
     public void onDraw(Canvas canvas) {
         super.onDraw(canvas);
+        if (mElipse.isUndefined()) {
+            setRepresentation(mVignetteRep);
+        }
         mElipse.draw(canvas);
 
     }