aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaxime Biais <max@bia.is>2016-09-26 15:40:17 +0200
committerGitHub <noreply@github.com>2016-09-26 15:40:17 +0200
commitbb3667f1889a5c4ba6d776c04e6b658cf717e5f5 (patch)
treebcac4f743052a833cf95a416ed596989524b0742
parent41d4b84841af27785deb35f0d9b0e42850471017 (diff)
parent4c27ae38fd9410d4b211296a6fd985a4d8ac35e7 (diff)
downloadgradle-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.java6
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));
+ }
}
}