am 1ed017ef: Fix performance issue when there are no contacts in the dictionary dictionary.

* commit '1ed017ef0e271ed3f3c212def6cc6ba95b14e780':
  Fix performance issue when there are no contacts in the dictionary dictionary.
main
Tom Ouyang 2012-05-29 10:09:29 -07:00 committed by Android Git Automerger
commit 126ed42415
2 changed files with 34 additions and 12 deletions

View File

@ -214,6 +214,10 @@ public class ContactsBinaryDictionary extends ExpandableBinaryDictionary {
return false; return false;
} }
if (contactCount != sContactCountAtLastRebuild) { if (contactCount != sContactCountAtLastRebuild) {
if (DEBUG) {
Log.d(TAG, "Contact count changed: " + sContactCountAtLastRebuild + " to "
+ contactCount);
}
return true; return true;
} }
// Check all contacts since it's not possible to find out which names have changed. // Check all contacts since it's not possible to find out which names have changed.

View File

@ -294,7 +294,7 @@ abstract public class ExpandableBinaryDictionary extends Dictionary {
*/ */
protected void loadBinaryDictionary() { protected void loadBinaryDictionary() {
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "Loading binary dictionary: request=" Log.d(TAG, "Loading binary dictionary: " + mFilename + " request="
+ mSharedDictionaryController.mLastUpdateRequestTime + " update=" + mSharedDictionaryController.mLastUpdateRequestTime + " update="
+ mSharedDictionaryController.mLastUpdateTime); + mSharedDictionaryController.mLastUpdateTime);
} }
@ -326,7 +326,7 @@ abstract public class ExpandableBinaryDictionary extends Dictionary {
*/ */
private void generateBinaryDictionary() { private void generateBinaryDictionary() {
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "Generating binary dictionary: request=" Log.d(TAG, "Generating binary dictionary: " + mFilename + " request="
+ mSharedDictionaryController.mLastUpdateRequestTime + " update=" + mSharedDictionaryController.mLastUpdateRequestTime + " update="
+ mSharedDictionaryController.mLastUpdateTime); + mSharedDictionaryController.mLastUpdateTime);
} }
@ -371,7 +371,7 @@ abstract public class ExpandableBinaryDictionary extends Dictionary {
mLocalDictionaryController.mLastUpdateRequestTime = time; mLocalDictionaryController.mLastUpdateRequestTime = time;
mSharedDictionaryController.mLastUpdateRequestTime = time; mSharedDictionaryController.mLastUpdateRequestTime = time;
if (DEBUG) { if (DEBUG) {
Log.d(TAG, "Reload request: request=" + time + " update=" Log.d(TAG, "Reload request: " + mFilename + ": request=" + time + " update="
+ mSharedDictionaryController.mLastUpdateTime); + mSharedDictionaryController.mLastUpdateTime);
} }
} }
@ -380,28 +380,46 @@ abstract public class ExpandableBinaryDictionary extends Dictionary {
* Reloads the dictionary if required. Reload will occur asynchronously in a separate thread. * Reloads the dictionary if required. Reload will occur asynchronously in a separate thread.
*/ */
void asyncReloadDictionaryIfRequired() { void asyncReloadDictionaryIfRequired() {
if (!isReloadRequired()) return;
if (DEBUG) {
Log.d(TAG, "Starting AsyncReloadDictionaryTask: " + mFilename);
}
new AsyncReloadDictionaryTask().start(); new AsyncReloadDictionaryTask().start();
} }
/** /**
* Reloads the dictionary if required. Access is controlled on a per dictionary file basis and * Reloads the dictionary if required.
* supports concurrent calls from multiple instances that share the same dictionary file.
*/ */
protected final void syncReloadDictionaryIfRequired() { protected final void syncReloadDictionaryIfRequired() {
if (mBinaryDictionary != null && !mLocalDictionaryController.isOutOfDate()) { if (!isReloadRequired()) return;
return; syncReloadDictionaryInternal();
} }
/**
* Returns whether a dictionary reload is required.
*/
private boolean isReloadRequired() {
return mBinaryDictionary == null || mLocalDictionaryController.isOutOfDate();
}
/**
* Reloads the dictionary. Access is controlled on a per dictionary file basis and supports
* concurrent calls from multiple instances that share the same dictionary file.
*/
private final void syncReloadDictionaryInternal() {
// Ensure that only one thread attempts to read or write to the shared binary dictionary // Ensure that only one thread attempts to read or write to the shared binary dictionary
// file at the same time. // file at the same time.
mSharedDictionaryController.lock(); mSharedDictionaryController.lock();
try { try {
final long time = SystemClock.uptimeMillis(); final long time = SystemClock.uptimeMillis();
if (mSharedDictionaryController.isOutOfDate() || !dictionaryFileExists()) { final boolean dictionaryFileExists = dictionaryFileExists();
if (mSharedDictionaryController.isOutOfDate() || !dictionaryFileExists) {
// If the shared dictionary file does not exist or is out of date, the first // If the shared dictionary file does not exist or is out of date, the first
// instance that acquires the lock will generate a new one. // instance that acquires the lock will generate a new one.
if (hasContentChanged()) { if (hasContentChanged() || !dictionaryFileExists) {
// If the source content has changed, rebuild the binary dictionary. // If the source content has changed or the dictionary does not exist, rebuild
// the binary dictionary. Empty dictionaries are supported (in the case where
// loadDictionaryAsync() adds nothing) in order to provide a uniform framework.
mSharedDictionaryController.mLastUpdateTime = time; mSharedDictionaryController.mLastUpdateTime = time;
generateBinaryDictionary(); generateBinaryDictionary();
loadBinaryDictionary(); loadBinaryDictionary();
@ -435,7 +453,7 @@ abstract public class ExpandableBinaryDictionary extends Dictionary {
private class AsyncReloadDictionaryTask extends Thread { private class AsyncReloadDictionaryTask extends Thread {
@Override @Override
public void run() { public void run() {
syncReloadDictionaryIfRequired(); syncReloadDictionaryInternal();
} }
} }