From 5619d26af83f861596a12075d35caba6e6fafd94 Mon Sep 17 00:00:00 2001 From: Alastor Wu Date: Thu, 6 Oct 2016 14:02:03 +0800 Subject: [PATCH] Bug 1305430 - part2 : replace action checking with isMediaPlaying(). r=sebastian MozReview-Commit-ID: F2WyASzWAMJ --HG-- extra : rebase_source : 07a9a41ab745907306baf420a747423d995ab9d4 --- .../java/org/mozilla/gecko/media/MediaControlService.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java b/mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java index ccf315a756bf..a9ee955807de 100644 --- a/mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java +++ b/mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java @@ -334,7 +334,7 @@ public class MediaControlService extends Service implements Tabs.OnTabsChangedLi final Notification.MediaStyle style = new Notification.MediaStyle(); style.setShowActionsInCompactView(0); - final boolean isMediaPlaying = action.equals(ACTION_PAUSE); + final boolean isPlaying = isMediaPlaying(); final int visibility = tab.isPrivate() ? Notification.VISIBILITY_PRIVATE : Notification.VISIBILITY_PUBLIC; @@ -347,13 +347,13 @@ public class MediaControlService extends Service implements Tabs.OnTabsChangedLi .setDeleteIntent(createDeleteIntent()) .setStyle(style) .addAction(createNotificationAction(action)) - .setOngoing(isMediaPlaying) + .setOngoing(isPlaying) .setShowWhen(false) .setWhen(0) .setVisibility(visibility) .build(); - if (isMediaPlaying) { + if (isPlaying) { startForeground(MEDIA_CONTROL_ID, notification); } else { stopForeground(false);