GNOME Bugzilla – Bug 662031
media-export: Deactivate on slow Tracker start-up
Last modified: 2011-10-20 17:08:28 UTC
Automatic shutdown was not working under some circumstances
Created attachment 199257 [details] [review] media-export: Deactivate on slow Tracker start-up If the startup of the Tracker plug-in is delayed (e.g. due to DBus activation) the old method of disabling MediaExport did not work since the Tracker plug-in was not active when MediaExport was loaded. The plug-in now listens to notify::active of the Tracker plug-in and deactivates itself once that changes to "true".
Review of attachment 199257 [details] [review]: Looks good otherwise. ::: src/plugins/media-export/rygel-media-export-plugin.vala @@ +66,3 @@ } +public void shutdown_media_export () { this should be private and at the bottom?
Created attachment 199311 [details] [review] media-export: Deactivate on slow Tracker start-up If the startup of the Tracker plug-in is delayed (e.g. due to DBus activation) the old method of disabling MediaExport did not work since the Tracker plug-in was not active when MediaExport was loaded. The plug-in now listens to notify::active of the Tracker plug-in and deactivates itself once that changes to "true".
Review of attachment 199311 [details] [review]: Great!
The following fix has been pushed: 1f2f3fd media-export: Deactivate on slow Tracker start-up
Created attachment 199556 [details] [review] media-export: Deactivate on slow Tracker start-up If the startup of the Tracker plug-in is delayed (e.g. due to DBus activation) the old method of disabling MediaExport did not work since the Tracker plug-in was not active when MediaExport was loaded. The plug-in now listens to notify::active of the Tracker plug-in and deactivates itself once that changes to "true".