Use the same function for expand all applications

[ren2r]

In plasma-desktop the gtk menus are already exported to gmenu_dbusmenu_proxy, and use the same methods as qt applications and we can unify the code. Some apps like kwrite has deep menu items that only appears if we call About ToShow at least once for each submenu (e.g. highlighting, mode ) ... this way the method expanse_all_menu_with_dbus was not working since it would need a loop to expand all the submenus.

[Zren]

The main reason we kept the GTK path was because it's so much faster than the KDE path. I'll need to compare the recursive KDE path speed vs GTK before making a final decision.

I can merge it into performance, but I will be manually breaking this patch into 4-5 smaller commits for the master branch. You've got a lot of good stuff in this patch.

• You've also removed the sorting of the menuKeys, leaving them in insertion order (which is a good idea).
• The get_layout function is a good idea. dbusmenu_object_iface.GetLayout() returns [revision, layout] btw, which is why we have the magic [1] index.
• I'm going to reverse the if ___ not in ___: since there is an else statement block as well. It's always better to use if true: else: rather than if false: else:.
• You don't send dbus.UInt32(time.time()) in the Firefox "opened" event, was it not necessary?

TODO: Force push to ammend the commits in the performance-pr16 branch to give you credits before a final merge to master.

See the comparison: https://github.com/Zren/plasma-hud/compare/master...performance-pr16