From e5465d339fd64f7c07d776ebb3ecaf46640b2df1 Mon Sep 17 00:00:00 2001
From: Ruben Brunk <rubenbrunk@google.com>
Date: Thu, 13 Dec 2012 21:55:53 -0800
Subject: Fix Seekbar infinite recursion.

Bug: 7739384

Change-Id: Ie76d6741be4d08f3f9919662f1ad558c809aa5bd
---
 .../gallery3d/filtershow/imageshow/ImageShow.java       | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

(limited to 'src/com/android/gallery3d/filtershow/imageshow/ImageShow.java')

diff --git a/src/com/android/gallery3d/filtershow/imageshow/ImageShow.java b/src/com/android/gallery3d/filtershow/imageshow/ImageShow.java
index d9df7b7fc..f490859c9 100644
--- a/src/com/android/gallery3d/filtershow/imageshow/ImageShow.java
+++ b/src/com/android/gallery3d/filtershow/imageshow/ImageShow.java
@@ -153,15 +153,15 @@ public class ImageShow extends View implements OnGestureListener,
     private final Handler mHandler = new Handler();
 
     public void select() {
+        if (mSeekBar != null) {
+            mSeekBar.setOnSeekBarChangeListener(this);
+        }
         if (getCurrentFilter() != null) {
             int parameter = getCurrentFilter().getParameter();
             int maxp = getCurrentFilter().getMaxParameter();
             int minp = getCurrentFilter().getMinParameter();
             updateSeekBar(parameter, minp, maxp);
         }
-        if (mSeekBar != null) {
-            mSeekBar.setOnSeekBarChangeListener(this);
-        }
     }
 
     private int parameterToUI(int parameter, int minp, int maxp, int uimax) {
@@ -179,9 +179,6 @@ public class ImageShow extends View implements OnGestureListener,
         int seekMax = mSeekBar.getMax();
         int progress = parameterToUI(parameter, minp, maxp, seekMax);
         mSeekBar.setProgress(progress);
-        if (getPanelController() != null) {
-            getPanelController().onNewValue(parameter);
-        }
     }
 
     public void unselect() {
@@ -198,7 +195,8 @@ public class ImageShow extends View implements OnGestureListener,
     public void resetParameter() {
         ImageFilter currentFilter = getCurrentFilter();
         if (currentFilter != null) {
-            onNewValue(currentFilter.getDefaultParameter());
+            updateSeekBar(currentFilter.getDefaultParameter(),
+                    getCurrentFilter().getMinParameter(), getCurrentFilter().getMaxParameter());
         }
         if (USE_SLIDER_GESTURE) {
             mSliderController.reset();
@@ -215,8 +213,8 @@ public class ImageShow extends View implements OnGestureListener,
 
     @Override
     public void onNewValue(int parameter) {
-        int maxp = 100;
-        int minp = -100;
+        int maxp = ImageFilter.DEFAULT_MAX_PARAMETER;
+        int minp = ImageFilter.DEFAULT_MIN_PARAMETER;
         if (getCurrentFilter() != null) {
             getCurrentFilter().setParameter(parameter);
             maxp = getCurrentFilter().getMaxParameter();
@@ -229,7 +227,6 @@ public class ImageShow extends View implements OnGestureListener,
         if (getPanelController() != null) {
             getPanelController().onNewValue(parameter);
         }
-        updateSeekBar(parameter, minp, maxp);
         invalidate();
         mActivity.enableSave(hasModifications());
     }
-- 
cgit v1.2.3