diff options
author | junjiez <junjiez@codeaurora.org> | 2016-09-07 11:27:18 +0800 |
---|---|---|
committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2016-09-14 17:22:52 -0700 |
commit | dbfa594f2b588c8b52e87eb5f90c0df05bde6a7e (patch) | |
tree | ad28d02c3985cca3b5452e443165a23eba18cd21 /src/com/android/camera/SettingsManager.java | |
parent | e2c6bdcd24425d811cf98b967a09a42449967f56 (diff) | |
download | android_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.java | 20 |
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(); |