[banshee] NotificationArea: refactoring, extract method

[Andrés Aragoneses <aaragoneses src gnome org>]

commit 4a160e573081af67bda0864690db44d80572d059
Author: Andrés G. Aragoneses <knocte gmail com>
Date:   Tue Jul 22 02:10:14 2014 +0200

    NotificationArea: refactoring, extract method

 .../NotificationAreaService.cs                     |   37 +++++++++++--------
 1 files changed, 21 insertions(+), 16 deletions(-)
---
diff --git a/src/Extensions/Banshee.NotificationArea/Banshee.NotificationArea/NotificationAreaService.cs 
b/src/Extensions/Banshee.NotificationArea/Banshee.NotificationArea/NotificationAreaService.cs
index 4e98a10..dc6c0f0 100644
--- a/src/Extensions/Banshee.NotificationArea/Banshee.NotificationArea/NotificationAreaService.cs
+++ b/src/Extensions/Banshee.NotificationArea/Banshee.NotificationArea/NotificationAreaService.cs
@@ -491,22 +491,7 @@ namespace Banshee.NotificationArea
                 current_nf.Urgency = Urgency.Low;
                 current_nf.Timeout = 4500;
 
-                if (!current_track.IsLive && ActionsSupported && 
interface_action_service.PlaybackActions["NextAction"].Sensitive) {
-                    if (ActionIconsSupported) {
-                        current_nf.AddHint ("action-icons", true);
-
-                        // We need to use an icon name as the action id, so that the notification uses that 
icon
-                        current_nf.AddAction ("media-skip-backward",
-                            Catalog.GetString("Previous"), OnPreviousTrack);
-
-                        bool is_playing = ServiceManager.PlayerEngine.IsPlaying ();
-                        current_nf.AddAction (is_playing ? "media-playback-pause" : "media-playback-start",
-                            interface_action_service.PlaybackActions["PlayPauseAction"].Label, OnPlayPause);
-                    }
-
-                    current_nf.AddAction ("media-skip-forward",
-                        Catalog.GetString("Skip this item"), OnNextTrack);
-                }
+                UpdateActions ();
 
                 if (image == null) {
                     current_nf.RemoveHint ("image-path");
@@ -524,6 +509,26 @@ namespace Banshee.NotificationArea
             }
         }
 
+        private void UpdateActions ()
+        {
+            if (!current_track.IsLive && ActionsSupported && 
interface_action_service.PlaybackActions["NextAction"].Sensitive) {
+                if (ActionIconsSupported) {
+                    current_nf.AddHint ("action-icons", true);
+
+                    // We need to use an icon name as the action id, so that the notification uses that icon
+                    current_nf.AddAction ("media-skip-backward",
+                                          Catalog.GetString("Previous"), OnPreviousTrack);
+
+                    bool is_playing = ServiceManager.PlayerEngine.IsPlaying ();
+                    current_nf.AddAction (is_playing ? "media-playback-pause" : "media-playback-start",
+                                          interface_action_service.PlaybackActions["PlayPauseAction"].Label, 
OnPlayPause);
+                }
+
+                current_nf.AddAction ("media-skip-forward",
+                                      Catalog.GetString("Skip this item"), OnNextTrack);
+            }
+        }
+
         private string MarkupFormat (string fmt, params string [] args)
         {
             string [] new_args = new string [args.Length + 2];