From 0842797a25af0383b25b039f8d5436d861134157 Mon Sep 17 00:00:00 2001 From: Jason Monk Date: Thu, 21 Apr 2016 14:15:13 -0400 Subject: [PATCH] Fix summary loader lifecycle Previous CL accidentally cleared the listening bit too early making it never actually tell the SummaryProviders to stop listening. Change-Id: I6af1acc861f6987b33641c40637354c75cf4d708 Fixes: 28306872 --- src/com/android/settings/dashboard/SummaryLoader.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/com/android/settings/dashboard/SummaryLoader.java b/src/com/android/settings/dashboard/SummaryLoader.java index 780d68e0b67..13f2cf48740 100644 --- a/src/com/android/settings/dashboard/SummaryLoader.java +++ b/src/com/android/settings/dashboard/SummaryLoader.java @@ -97,7 +97,6 @@ public class SummaryLoader { public void setListening(boolean listening) { synchronized (mReceivers) { // Unregister listeners immediately. - mListening = false; for (int i = 0; i < mReceivers.size(); i++) { mActivity.unregisterReceiver(mReceivers.get(i)); } @@ -163,7 +162,11 @@ public class SummaryLoader { if (DEBUG) Log.d(TAG, "Listening " + listening); mListening = listening; for (SummaryProvider p : mSummaryMap.keySet()) { - p.setListening(listening); + try { + p.setListening(listening); + } catch (Exception e) { + Log.d(TAG, "Problem in setListening", e); + } } }