GNOME Bugzilla – Bug 697560
shellEntry: Remove close method
Last modified: 2013-04-08 19:30:54 UTC
Fixes a critical (lt-gnome-shell-real:26033): Clutter-CRITICAL **: clutter_actor_grab_key_focus: assertion `CLUTTER_IS_ACTOR (self)' failed with the following trace: 0 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/shellEntry.js:86 1 wrapper()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/lang.js:213 2 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/popupMenu.js:1160 3 wrapper()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/lang.js:213 4 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/shellEntry.js:176 5 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/modalDialog.js:116 6 wrapper()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/lang.js:213 7 _parent()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/lang.js:175 8 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/unlockDialog.js:313 9 wrapper()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/lang.js:213 10 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/screenShield.js:1029 11 wrapper()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/lang.js:213 12 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/screenShield.js:954 13 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/tweener.js:113 14 _callOnFunction()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/tweener/tweener.js:202 15 _updateTweenByIndex()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/tweener/tweener.js:332 16 _updateTweens()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/tweener/tweener.js:344 17 _onEnterFrame()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/tweener/tweener.js:359 18 _emit("name" = undefined, "handlers" = undefined, "i" = undefined, "length" = undefined, "arg_array" = undefined, "arguments" = undefined)@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/signals.js:124 19 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/tweener.js:253 20 wrapper()@/home/rmatos/Source/git.gnome.org/install/share/gjs-1.0/lang.js:213 21 anonymous()@/home/rmatos/Source/git.gnome.org/gnome-shell/js/ui/tweener.js:228
Created attachment 240951 [details] [review] shellEntry: Remove close method GrabHelper already takes care of putting the key focus back on the widget that had it when the grab was established. ShellEntry's close() has recently become harmful since it's now called from the menu's destroy() method and that is called in the entry's destroy handler which means that this._entry might no longer be valid at close() time.
Review of attachment 240951 [details] [review]: OK.
Attachment 240951 [details] pushed as 07053c3 - shellEntry: Remove close method