GNOME Bugzilla – Bug 681539
shell: also destroy ignored items
Last modified: 2012-11-21 22:22:54 UTC
Created attachment 220812 [details] [review] shell: also destroy ignored items Fix a loop introduced while fixing the async add items. Appears when empathy contact list disappear (crash ?), the empathy-chat irc window triggers a loop into gnome-shell and a click on the application area in gnome-shell shows a list containing empty placeholders (except for the section titles), Patch that fixes the similar issue when adding items by me introduced this new issue (as I extended it too broadly to item removal). This one revert the removal part to its previous state.
Review of attachment 220812 [details] [review]: currentItems.length includes all items (since it is derived from _getMenuItems()). How the change corresponds to the commit subject?
indeed I encountered the issue again (ie gnome-shell loop while empathy contact list disappear and empath-chat window stays with an application menu only containy in the popupmenu the submenu headers for chatrooms and contacts). this issue thus is not fixed by the above change and needs more digging. Thanks for the check. please do not commit. I will attempt to find the root of this bug (even if the context is hard to reproduce). If anybody manage to reproduce reliably the bug please add to the report. (ie I wonder if this only happens if empathy contact window crash or else)
Finally I found a way to reproduce in a reliable way. That s a start. with empathy roster + empathy chat + gnome shell. When I restart gnome shell (or it crashed and spawned) if I switch to the empathy chat window before the roster one , I get a nearly empty app menu and then if I switch back to the roster window , gnome-shell "freeze" . Ie it is not stuck but attempt really hard to build a menu but always get "action-added" from RemoteMenu._createMenuItem for most items.
Can you try the patch in #686502? It sounds like the same issue i've solved there
applying patch from #686502 , fixes the freeze . Appmenu is still nearly empty when switching to the chat window before the roster one after a gnome-shell restart. But this last issue is not of the same level (minor and probably for the empathy side of things). *** This bug has been marked as a duplicate of bug 686502 ***