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



commit 830c2950e2f7c48fe5416e45813ce6fabf9e67a1
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 c56ed76..bda9dce 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1303,7 +1303,7 @@ const Source = new Lang.Class({
 
         this.notifications.splice(index, 1);
         if (this.notifications.length == 0)
-            this._lastNotificationRemoved();
+            this.destroy();
 
         this.countUpdated();
     },
@@ -1356,11 +1356,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]