Fixed a memory leak in the bluetooth settings
Release resources in onPause rather than onResume to fix a memory leak. Without this fix the app will crash if you are in Bluetooth settings and tap "Device name" and then roate the device a few times. Change-Id: I57a8346247c13f5113e7c4ea39dde5b29a7359bb
This commit is contained in:
committed by
Johan Redestig
parent
e8531ff6a0
commit
00fa2f578e
@@ -161,8 +161,6 @@ public class BluetoothSettings extends PreferenceActivity
|
|||||||
|
|
||||||
// Repopulate (which isn't too bad since it's cached in the settings
|
// Repopulate (which isn't too bad since it's cached in the settings
|
||||||
// bluetooth manager
|
// bluetooth manager
|
||||||
mDevicePreferenceMap.clear();
|
|
||||||
mDeviceList.removeAll();
|
|
||||||
addDevices();
|
addDevices();
|
||||||
|
|
||||||
if (mScreenType == SCREEN_TYPE_SETTINGS) {
|
if (mScreenType == SCREEN_TYPE_SETTINGS) {
|
||||||
@@ -187,6 +185,9 @@ public class BluetoothSettings extends PreferenceActivity
|
|||||||
protected void onPause() {
|
protected void onPause() {
|
||||||
super.onPause();
|
super.onPause();
|
||||||
|
|
||||||
|
mDevicePreferenceMap.clear();
|
||||||
|
mDeviceList.removeAll();
|
||||||
|
|
||||||
mLocalManager.setForegroundActivity(null);
|
mLocalManager.setForegroundActivity(null);
|
||||||
mDevicePreferenceMap.clear();
|
mDevicePreferenceMap.clear();
|
||||||
mDeviceList.removeAll();
|
mDeviceList.removeAll();
|
||||||
|
Reference in New Issue
Block a user