[gnome-shell/wip/fmuellner/notification-redux+sass: 87/122] messageTray: Remove _lastNotificationRemoved()



commit 0a55972ea9074e95fbdce1c4b13954e22e4d7436
Author: Florian Müllner <fmuellner gnome org>
Date:   Sun Feb 15 20:19:03 2015 +0100

    messageTray: Remove _lastNotificationRemoved()
    
    No Source subclass used it to do anything special, and with sources
    no longer having any UI representation on their own, doing anything
    else isn't useful either.

 js/ui/messageTray.js        |    7 +------
 js/ui/notificationDaemon.js |    4 ----
 2 files changed, 1 insertions(+), 10 deletions(-)
---
diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 8bf066f..65d07c4 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1136,7 +1136,7 @@ const Source = new Lang.Class({
 
         this.notifications.splice(index, 1);
         if (this.notifications.length == 0)
-            this._lastNotificationRemoved();
+            this.destroy();
 
         this.countUpdated();
     },
@@ -1198,11 +1198,6 @@ const Source = new Lang.Class({
                 this.notifications[i].destroy();
 
         this.countUpdated();
-    },
-
-    // Default implementation is to destroy this source, but subclasses can override
-    _lastNotificationRemoved: function() {
-        this.destroy();
     }
 });
 Signals.addSignalMethods(Source.prototype);
diff --git a/js/ui/notificationDaemon.js b/js/ui/notificationDaemon.js
index 8b97607..31c0726 100644
--- a/js/ui/notificationDaemon.js
+++ b/js/ui/notificationDaemon.js
@@ -582,10 +582,6 @@ const FdoNotificationDaemonSource = new Lang.Class({
         this.destroyNonResidentNotifications();
     },
 
-    _lastNotificationRemoved: function() {
-        this.destroy();
-    },
-
     openApp: function() {
         if (this.app == null)
             return;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]