diff options
author | Taesu Lee <taesu82.lee@samsung.com> | 2020-05-15 20:09:24 +0900 |
---|---|---|
committer | Michael Bestas <mkbestas@lineageos.org> | 2020-05-24 20:13:50 +0300 |
commit | bdacda42b5d6b3a6ad4ae17abfd903fe14c83691 (patch) | |
tree | 92ada10dc1b2164bd4854918196a39c941f888fd /src/com/android/messaging/datamodel/action/ProcessPendingMessagesAction.java | |
parent | 398ad51836825699c01e13c1d6077ee80d904dba (diff) | |
download | packages_apps_Messaging-bdacda42b5d6b3a6ad4ae17abfd903fe14c83691.tar.gz packages_apps_Messaging-bdacda42b5d6b3a6ad4ae17abfd903fe14c83691.tar.bz2 packages_apps_Messaging-bdacda42b5d6b3a6ad4ae17abfd903fe14c83691.zip |
Rescheduling pending messages after outside window failure
No more rescheduling for pending messages after DownloadMmsAction is
failed caused by outside window. Any pending download messages will be
remained as the DOWNLOADING state without actual downloading operations
and sending a DEFERRED M-NotifyResp.ind even.
It will reschedule pending messages whenever sending or downloading is
failed including outside window failure case. And it also removes dup
scheduling and fix some typo.
Test: Manual
Change-Id: Ie37088347c69f25d838875830fc65ce7727fefe6
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Diffstat (limited to 'src/com/android/messaging/datamodel/action/ProcessPendingMessagesAction.java')
-rw-r--r-- | src/com/android/messaging/datamodel/action/ProcessPendingMessagesAction.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/com/android/messaging/datamodel/action/ProcessPendingMessagesAction.java b/src/com/android/messaging/datamodel/action/ProcessPendingMessagesAction.java index f164148..1a3eb63 100644 --- a/src/com/android/messaging/datamodel/action/ProcessPendingMessagesAction.java +++ b/src/com/android/messaging/datamodel/action/ProcessPendingMessagesAction.java @@ -301,12 +301,16 @@ public class ProcessPendingMessagesAction extends Action implements Parcelable { unregister(subId); if (PhoneUtils.getDefault().isDefaultSmsApp()) { - queueActions(this); + if (!queueActions(this)) { + LogUtil.v(TAG, "ProcessPendingMessagesAction: rescheduling"); + // TODO: Need to clear retry count here? + scheduleProcessPendingMessagesAction(true /* failed */, this); + } } else { if (LogUtil.isLoggable(TAG, LogUtil.VERBOSE)) { LogUtil.v(TAG, "ProcessPendingMessagesAction: Not default SMS app; rescheduling"); } - scheduleProcessPendingMessagesAction(true, this); + scheduleProcessPendingMessagesAction(true /* failed */, this); } return null; |