summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMatt Garnes <matt@cyngn.com>2015-03-06 11:31:45 -0800
committerSteve Kondik <steve@cyngn.com>2015-03-22 15:03:04 -0700
commit204848fa248948c60757fe7823c73980cc85cf9a (patch)
tree307d09a0e9b169f0175d676a1c7b07bc73deab91 /src
parent0c4b0db4b2932833e9f1ea727089aae6fee2b41e (diff)
downloadandroid_packages_apps_ContactsCommon-204848fa248948c60757fe7823c73980cc85cf9a.tar.gz
android_packages_apps_ContactsCommon-204848fa248948c60757fe7823c73980cc85cf9a.tar.bz2
android_packages_apps_ContactsCommon-204848fa248948c60757fe7823c73980cc85cf9a.zip
Only reload contacts if relevant SharedPreferences change.
In ContactEntryListFragment, we listen for any changes to SharedPreferences. However, not all SharedPreferences should trigger a data reload. last_updated_millis will be written if the smart dial tables in the dialer DB were updated. mPreferencesChangeListener would reload all contacts data. This shared preference does not change the data for contacts, so we should not reload them. On slower devices, this will visibly cause the ListView to flash between the empty view and populated listitems as it reloads twice when the activity resumes. Only reload data when relevant SharedPreferences change. Change-Id: Id2a607fdaf5715b69738c76422ef08d76e6ffafe
Diffstat (limited to 'src')
-rwxr-xr-xsrc/com/android/contacts/common/list/ContactEntryListFragment.java5
1 files changed, 3 insertions, 2 deletions
diff --git a/src/com/android/contacts/common/list/ContactEntryListFragment.java b/src/com/android/contacts/common/list/ContactEntryListFragment.java
index 4af18305..1819c0c8 100755
--- a/src/com/android/contacts/common/list/ContactEntryListFragment.java
+++ b/src/com/android/contacts/common/list/ContactEntryListFragment.java
@@ -916,8 +916,9 @@ public abstract class ContactEntryListFragment<T extends ContactEntryListAdapter
new ContactsPreferences.ChangeListener() {
@Override
public void onChange() {
- loadPreferences();
- reloadData();
+ if(loadPreferences()) {
+ reloadData();
+ }
}
};