← Back to team overview

ayatana-commits team mailing list archive

[Merge] lp:~ted/indicator-session/restart-icon-change into lp:indicator-session

 

Ted Gould has proposed merging lp:~ted/indicator-session/restart-icon-change into lp:indicator-session.

Requested reviews:
  Indicator Applet Developers (indicator-applet-developers)
Related bugs:
  Bug #854292 in Session Menu: "indicator-session power menu disappears after running update-manager or apt-get"
  https://bugs.launchpad.net/indicator-session/+bug/854292

For more details, see:
https://code.launchpad.net/~ted/indicator-session/restart-icon-change/+merge/77090

Fixes the icon to update instead of replacing.  This probably caused an
interesting race in the panel service making the panel service think it was
hidden in some cases.  Depends on when it updated.
-- 
https://code.launchpad.net/~ted/indicator-session/restart-icon-change/+merge/77090
Your team ayatana-commits is subscribed to branch lp:indicator-session.
=== modified file 'src/indicator-session.c'
--- src/indicator-session.c	2011-09-08 17:40:07 +0000
+++ src/indicator-session.c	2011-09-27 04:33:18 +0000
@@ -509,11 +509,11 @@
   }
   else if (g_strcmp0(signal_name, "RestartRequired") == 0) {    
     if (greeter_mode == TRUE){
-      self->devices.image = indicator_image_helper (GREETER_ICON_RESTART);
+      indicator_image_helper_update(self->devices.image, GREETER_ICON_RESTART);
     }
     else{
       g_debug ("reboot required");
-      self->devices.image = indicator_image_helper (ICON_RESTART);      
+      indicator_image_helper_update(self->devices.image, ICON_RESTART);
     }
   }  
 }


Follow ups