Submenus not added when done so with Glade

Further testing confirmed this is an issue with libappindicator. I'm trying to fix this by listening to notify::submenu.

Jorge told to to assign this bug to Aurélien Gâteau, but I forgot to do so and tried to find a solution myself.
I'm now assigning this bug to Aurélien because I'm not sure if my proposed fixed in lp~sense/indicator-application/submenu-later is good enough since I can't fully test it myself due to problems with my local installation.
Aurélien, would you mind taking a look?

The 'child-added' signal, which is currently the signal used by libappindicator to check for newly added submenus is only emitted when a GtkMenuItem is insterted in a GtkMenuShell. This means we need to add an extra signal for submenus added using gtk_menu_item_set_submenu(), because that is what we need to watch new submenus.
I'm working on a patch to GTK+2.0.

Attached a branch with a partial fix for AppInd's part of the problem, at lp:~sense/indicator-application/fix-608219
I'm pushing this non-working branch because I'll be away on a two-week holiday starting tomorrow, and now someone else can work on this bug.

The problem is that when you launch Deluge, patched with the patch attached to bug #584669, it will segfault. In the syslog I find entries like these when that happens: "nott kernel: [31724.397769] /usr/local/bin/[7178]: segfault at 1 ip 00007f174a4bc3ce sp 00007fff75791940 error 4 in libdbusmenu-glib.so.1.0.12[7f174a4b7000+10000]".

This bug was fixed in the package gtk+2.0 - 2.21.5-1ubuntu6

---------------
gtk+2.0 (2.21.5-1ubuntu6) maverick; urgency=low

  * debian/patches/043_ubuntu_menu_proxy.patch:
    - update by Cody Russell required to get the mnemonic activation working
      in the indicator-appmenu.
  * debian/patches/090_git_calendar_style_properties.patch:
    - git change to allow tweaking GtkCalendar layouts styling
  * debian/libgtk2.0-0.symbols: updated for the previous changes

  [ Sense Egbert Hofstede ]
  * debian/patches/072_indicator_menu_update.patch: Adding 'submenu-added' and
    'submenu-removed' signals so Application Indicators now that. (lp: #608219)
 -- Sebastien Bacher <email address hidden> Wed, 04 Aug 2010 15:59:31 +0200

This bug was fixed in the package indicator-application - 0.2.9-0ubuntu1

---------------
indicator-application (0.2.9-0ubuntu1) maverick; urgency=low

  [ Ted Gould ]
  * New upstream release.
    * Fix Mono binding versioning and build (LP: #636132)
    * Fix submenus with Glade applications (LP: #608219)
    * Add new functions to Mono and Python bindings
  * debian/libappindicator0.1-cil.installcligac: Add the 0.1
    policy file.

  [ Didier Roche ]
  * debian/control:
    - build with latest libindicator-dev for ABI breakage (LP: #637692)
 -- Didier Roche <email address hidden> Wed, 22 Sep 2010 20:01:04 +0200

Because I unfortunately and by accident used a branch associated with this bug to fix another issue it was thought that this bug was fixed. It was not, and therefore I am now reopening the bug.
Apologises for the inconvenience.

Could somebody upstream the gtk patch added there?

https://bugs.launchpad.net/ayatana-ubuntu/+bug/584669 is affected by this bug, this is *partly* worked around by calling appindicator.set_menu() *AFTER* submenus have been appended to the menu.

However, a bug still exists in that the wrong widget is passed to the callback and therefore the setting is always read from the currently selected item rather than the activated item.

is that still an issue in natty? lot of indicator bugs got fixed there

Deluge added its own indicator meanwhile, and that is using the unedited tray_menu from the GtkStatusIcon. However, the submenus are not displayed at all here, so this seems to be an issue still.

They are using Glade, maybe that worsens the situation.

(The GTK+ patch is not used anymore. I'm not sure what is using it, but since Deluge isn't, it seems to me it can be removed again, unless someone sees a use case for it.)

@Ted: not sure what's left to fix, but after the big refactoring, it's worth checking if the problem is definitely fixed or not.

I'm going to mark this as incomplete. Can someone test it with current Natty please? Thanks.

We are closing this bug report as it lacks the information, described in the previous comments, we need to investigate the problem further. However, please reopen it if you can give us the missing information and don't hesitate to submit bug reports in the future.