summaryrefslogtreecommitdiffstats
path: root/src/com/android/camera/PhotoMenu.java
diff options
context:
space:
mode:
authorJack Yoo <jyoo@codeaurora.org>2016-01-27 17:09:15 -0800
committerCamera Software Integration <camswint@localhost>2016-01-30 13:01:11 -0700
commite94a646b72e995ec12b6b9ab5bd3212b3f6e299c (patch)
treef3f5c386467f56157f2fb5970d601c3051822fdc /src/com/android/camera/PhotoMenu.java
parent8f30e9a0e0d0a1c55d3409877b5a188885cf5887 (diff)
downloadandroid_packages_apps_Snap-e94a646b72e995ec12b6b9ab5bd3212b3f6e299c.tar.gz
android_packages_apps_Snap-e94a646b72e995ec12b6b9ab5bd3212b3f6e299c.tar.bz2
android_packages_apps_Snap-e94a646b72e995ec12b6b9ab5bd3212b3f6e299c.zip
SnapdragonCamera: Fix the memory leak on view.
Keep using modules instead of creating. Don't remove the view without cleaning it. Change-Id: I260a660cac3d770f7d02dfdc1881e15490da66a0 CRs-Fixed: 958510
Diffstat (limited to 'src/com/android/camera/PhotoMenu.java')
-rw-r--r--src/com/android/camera/PhotoMenu.java13
1 files changed, 2 insertions, 11 deletions
diff --git a/src/com/android/camera/PhotoMenu.java b/src/com/android/camera/PhotoMenu.java
index 0e4db9503..22cd8823f 100644
--- a/src/com/android/camera/PhotoMenu.java
+++ b/src/com/android/camera/PhotoMenu.java
@@ -1087,16 +1087,7 @@ public class PhotoMenu extends MenuController
.findPreference(CameraSettings.KEY_COLOR_EFFECT);
if (pref == null || pref.getValue() == null)
return;
-
- int[] iconIds = pref.getLargeIconIds();
- int index = pref.findIndexOfValue(pref.getValue());
- int resid = -1;
- if (!pref.getUseSingleIcon() && iconIds != null) {
- resid = iconIds[index];
- } else {
- resid = pref.getSingleIcon();
- }
- ((ImageView) button).setImageResource(resid);
+ changeFilterModeControlIcon(pref.getValue());
button.setVisibility(View.VISIBLE);
button.setOnClickListener(new OnClickListener() {
@Override
@@ -1450,8 +1441,8 @@ public class PhotoMenu extends MenuController
} else {
mHdrSwitcher.setVisibility(View.VISIBLE);
}
-
updateFilterModeIcon(pref, pref);
+
super.onSettingChanged(pref);
}