[Pharo-project] Issue 3550 in pharo: MenuItemMorph>>mouseDown: has unnecessary check

[pharo]

Status: New
Owner: ----

New issue 3550 by sean.p.d...@gmail.com: MenuItemMorph>>mouseDown: has  
unnecessary check
http://code.google.com/p/pharo/issues/detail?id=3550
Pharo1.2rc1 Latest update: #12310

If a menu item is clicked while it's pinned-down menu is obscured, the  
following code brings the menu to the front without executing the item's  
command:

        "Quick hack to bring menu to the front if it is obscured. See
		http://bugs.squeak.org/view.php?id=1780"; "fixed by gvc to work with  
embedded menus"
"  self owner owner = self world ifTrue: [
                self world morphsInFrontOf: self owner overlapping: self owner 
bounds
                        do: [:ignored | ^ self owner comeToFront]]."

However, in 1.2, it does not seem that a pinned-down menu /can/ be obscured  
- it magically stays on top.  So this check seems redundant.