Merge lp://staging/~john-pankov/dockmanager/quicklists into lp://staging/dockmanager

Since this is Ubuntu-specific, I think it'd be much better to read just the desktop cache file and also install a file monitor on it, and if it doesn't exist then assume this ain't Ubuntu and there are no Unity-specific desktop files.

AFAIK desktop*.cache doesn't contain X-Ayatana-Desktop-Shortcuts entries.
Also not sure if the file monitor is needed - we'll get item_path_found() if user launches newly installed app.
Please let me know your thoughts on this.

You're right, my bad... Still I don't like the crawling of all the desktop files, ultimately you do get the path to desktop file in the item_path_found() method, so it'd be much better to just load it there.

Interesting, but indeed you should use the desktop-file-path over dbus and scan it for quicklists. Scanning the whole applications-folder everytime isnt nice.

Fixed

1) the DesktopFile property can be empty string if the window can't be matched, you don't handle that
2) you should cache the desktop files you already loaded at some point, we don't need extra i/o on every window close/open cycle

Fixed both points

One last thing, please use glib/gio to launch the desktop files (instead of Popen), that will enable you to get rid of the remove_params function.

Fixed

Looks good to me...

Would be nice if available translations of the QuickList entries could be taken into account. This means using Name[LOCALE]=... with a fallback to Name=...

Done

...