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



commit 3de5c955f5c9c24e730ff22f729030a086dcd969
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, so just kill off that hook.

 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 4baed6a..4bf2580 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1304,7 +1304,7 @@ const Source = new Lang.Class({
 
         this.notifications.splice(index, 1);
         if (this.notifications.length == 0)
-            this._lastNotificationRemoved();
+            this.destroy();
 
         this.countUpdated();
     },
@@ -1357,11 +1357,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 aedda63..8d470c9 100644
--- a/js/ui/notificationDaemon.js
+++ b/js/ui/notificationDaemon.js
@@ -551,10 +551,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]