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



commit 3435d5a5b202129d70c6167eed53185400dbab01
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]