summaryrefslogtreecommitdiffstats
path: root/src/com/android/camera/SettingsManager.java
diff options
context:
space:
mode:
authorjunjiez <junjiez@codeaurora.org>2016-09-07 11:27:18 +0800
committerGerrit - the friendly Code Review server <code-review@localhost>2016-09-14 17:22:52 -0700
commitdbfa594f2b588c8b52e87eb5f90c0df05bde6a7e (patch)
treead28d02c3985cca3b5452e443165a23eba18cd21 /src/com/android/camera/SettingsManager.java
parente2c6bdcd24425d811cf98b967a09a42449967f56 (diff)
downloadandroid_packages_apps_Snap-dbfa594f2b588c8b52e87eb5f90c0df05bde6a7e.tar.gz
android_packages_apps_Snap-dbfa594f2b588c8b52e87eb5f90c0df05bde6a7e.tar.bz2
android_packages_apps_Snap-dbfa594f2b588c8b52e87eb5f90c0df05bde6a7e.zip
SnapdragonCamera:Fix instance of SettingsManager is null
When there are two CameraActivities, one of them is destroyed, the instance of settingsManager will be destroyed too. When the other CameraActivity invokes SettingsManager.getInstance(), it may return null and cause exceptions.Create and hold a instance of SettingsManager in each CameraActivity to make sure it's not null. Change-Id: Id2c355bf63f61e09f914113082db2876f28b9658 CRs-Fixed: 1060909
Diffstat (limited to 'src/com/android/camera/SettingsManager.java')
-rw-r--r--src/com/android/camera/SettingsManager.java20
1 files changed, 1 insertions, 19 deletions
diff --git a/src/com/android/camera/SettingsManager.java b/src/com/android/camera/SettingsManager.java
index c8f3f8c7b..f4162c79f 100644
--- a/src/com/android/camera/SettingsManager.java
+++ b/src/com/android/camera/SettingsManager.java
@@ -120,7 +120,6 @@ public class SettingsManager implements ListMenu.SettingsListener {
public static final String KEY_SHUTTER_SOUND = "pref_camera2_shutter_sound_key";
private static final String TAG = "SnapCam_SettingsManager";
- private static SettingsManager sInstance;
private ArrayList<CameraCharacteristics> mCharacteristics;
private ArrayList<Listener> mListeners;
private Map<String, Values> mValuesMap;
@@ -133,7 +132,7 @@ public class SettingsManager implements ListMenu.SettingsListener {
private JSONObject mDependency;
private int mCameraId;
- private SettingsManager(Context context) {
+ public SettingsManager(Context context) {
mListeners = new ArrayList<>();
mCharacteristics = new ArrayList<>();
mContext = context;
@@ -173,23 +172,6 @@ public class SettingsManager implements ListMenu.SettingsListener {
mDependency = parseJson("dependency.json");
}
- public static SettingsManager createInstance(Context context) {
- if (sInstance == null) {
- sInstance = new SettingsManager(context);
- }
- return sInstance;
- }
-
- public static SettingsManager getInstance() {
- return sInstance;
- }
-
- public void destroyInstance() {
- if (sInstance != null) {
- sInstance = null;
- }
- }
-
public List<String> getDisabledList() {
List<String> list = new ArrayList<>();
Set<String> keySet = mValuesMap.keySet();