diff options
author | Maxime Biais <max@bia.is> | 2016-09-26 15:40:17 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-09-26 15:40:17 +0200 |
commit | bb3667f1889a5c4ba6d776c04e6b658cf717e5f5 (patch) | |
tree | bcac4f743052a833cf95a416ed596989524b0742 | |
parent | 41d4b84841af27785deb35f0d9b0e42850471017 (diff) | |
parent | 4c27ae38fd9410d4b211296a6fd985a4d8ac35e7 (diff) | |
download | gradle-perf-android-medium-bb3667f1889a5c4ba6d776c04e6b658cf717e5f5.tar.gz |
Merge pull request #4563 from wordpress-mobile/issue/4556-gcm-update-notifs-on-system-bar-crash
Issue/4556 gcm update notifs on system bar crash
-rw-r--r-- | WordPress/src/main/java/org/wordpress/android/GCMMessageService.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/WordPress/src/main/java/org/wordpress/android/GCMMessageService.java b/WordPress/src/main/java/org/wordpress/android/GCMMessageService.java index 288ad522f..06ee0e38f 100644 --- a/WordPress/src/main/java/org/wordpress/android/GCMMessageService.java +++ b/WordPress/src/main/java/org/wordpress/android/GCMMessageService.java @@ -76,7 +76,7 @@ public class GCMMessageService extends GcmListenerService { private void synchronizedHandleDefaultPush(String from, @NonNull Bundle data) { // sActiveNotificationsMap being static, we can't just synchronize the method - synchronized (sActiveNotificationsMap) { + synchronized (GCMMessageService.class) { handleDefaultPush(from, data); } } @@ -407,7 +407,9 @@ public class GCMMessageService extends GcmListenerService { String noteType = StringUtils.notNullStr(remainingNote.getString(PUSH_ARG_TYPE)); String noteId = remainingNote.getString(PUSH_ARG_NOTE_ID, ""); - showIndividualNotificationForBuilder(builder, noteType, noteId, sActiveNotificationsMap.keyAt(0)); + if (!sActiveNotificationsMap.isEmpty()) { + showIndividualNotificationForBuilder(builder, noteType, noteId, sActiveNotificationsMap.keyAt(0)); + } } } |