Bug 1329902 - Part 3, don't notify Gecko if removed route is not a presentation display. r=kuoe0 draft
authorShih-Chiang Chien <schien@mozilla.com>
Wed, 11 Jan 2017 15:51:31 +0800
changeset 458878 9c9cb480bbcc320e1e6bc9613a3de2ab341de8db
parent 458877 2cedb3e4ea198bdd171ed16505c4fc1417640f3a
child 541771 1509cf3dc884064392a19c36f8e37ed64cdceefb
push id41087
push userschien@mozilla.com
push dateWed, 11 Jan 2017 07:52:27 +0000
reviewerskuoe0
bugs1329902
milestone53.0a1
Bug 1329902 - Part 3, don't notify Gecko if removed route is not a presentation display. r=kuoe0 MozReview-Commit-ID: 21kjmNYaW1b
mobile/android/base/java/org/mozilla/gecko/MediaPlayerManager.java
--- a/mobile/android/base/java/org/mozilla/gecko/MediaPlayerManager.java
+++ b/mobile/android/base/java/org/mozilla/gecko/MediaPlayerManager.java
@@ -182,18 +182,19 @@ public class MediaPlayerManager extends 
                 debug("onRouteRemoved: route=" + route);
 
                 // Remove from media player list.
                 players.remove(route.getId());
                 GeckoAppShell.notifyObservers("MediaPlayer:Removed", route.getId());
                 updatePresentation();
 
                 // Remove from presentation display list.
-                displays.remove(route.getId());
-                GeckoAppShell.notifyObservers("AndroidCastDevice:Removed", route.getId());
+                if (displays.remove(route.getId()) != null) {
+                    GeckoAppShell.notifyObservers("AndroidCastDevice:Removed", route.getId());
+                }
             }
 
             @Override
             public void onRouteSelected(MediaRouter router, MediaRouter.RouteInfo route) {
                 updatePresentation();
             }
 
             @Override