ADHOC
/
replicant-packages_apps_Email
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix threading issues in EmailDownloadService 

b/7257927

Change-Id: I390ced0eee18ff324032481ae55c31f5d4b26ffd
This commit is contained in:
Tony Mantler 2013-10-14 12:25:04 -07:00
parent 6df3709128
commit efd835acea
1 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/com/android/email/service/AttachmentDownloadService.java
View File

@ -921,7 +921,15 @@ public class AttachmentDownloadService extends Service implements Runnable {
// Loop until stopped, with a 30 minute wait loop
while (!mStop) {
// Here's where we run our attachment loading logic...
mConnectivityManager.waitForConnectivity();
// Make a local copy of the variable so we don't null-crash on service shutdown
final EmailConnectivityManager ecm = mConnectivityManager;
if (ecm != null) {
ecm.waitForConnectivity();
}
if (mStop) {
// We might be bailing out here due to the service shutting down
break;
}
mDownloadSet.processQueue();
if (mDownloadSet.isEmpty()) {
LogUtils.d(TAG, "*** All done; shutting down service");
@ -938,8 +946,10 @@ public class AttachmentDownloadService extends Service implements Runnable {
}
// Unregister now that we're done
if (mConnectivityManager != null) {
mConnectivityManager.unregister();
// Make a local copy of the variable so we don't null-crash on service shutdown
final EmailConnectivityManager ecm = mConnectivityManager;
if (ecm != null) {
ecm.unregister();
}
}
@ -975,6 +985,7 @@ public class AttachmentDownloadService extends Service implements Runnable {
}
if (mConnectivityManager != null) {
mConnectivityManager.unregister();
mConnectivityManager.stopWait();
mConnectivityManager = null;
}
}