commit 36c25af6a1b8fa2b9c5b0a3b5d419935fc9507b5
author mingliang.x.lei <mingliang.x.lei@sonymobile.com> Wed Mar 09 14:30:55 2016 +0800
committer Yoshinori Hirano <Yoshinori.Hirano@sonymobile.com> Wed Jul 12 05:42:14 2017 +0000
tree f57f5d64503b0ada00f2859f46b7c5073395a077
parent 3cbae7d6b5815825cdf171e36587c654af44ecbc

Fix memory leak in Bluetooth settings

Bluetooth settings screen has two categories. Each category should call
removeAll() method to unregister a callback from CachedBluetoothDevice.
But the method is not called for mPairedDevicesCategory.

Bug: 30004142
Test: manual - connect to Bluetooth device and then
               open and close Bluetooth settings screen repeatedly

Change-Id: I5a72994473ee2bb5fc3ad00176d9a930b4839099

src/com/android/settings/bluetooth/BluetoothSettings.java

diff --git a/src/com/android/settings/bluetooth/BluetoothSettings.java b/src/com/android/settings/bluetooth/BluetoothSettings.java
index c352e47..a8e56bf 100644
--- a/src/com/android/settings/bluetooth/BluetoothSettings.java
+++ b/src/com/android/settings/bluetooth/BluetoothSettings.java
@@ -210,6 +210,7 @@
     @Override
     public void onStop() {
         super.onStop();
+        mPairedDevicesCategory.removeAll();
         if (mBluetoothEnabler != null) {
             mBluetoothEnabler.pause();
         }