Comment #2 on issue 5156 by m0.interactive: Can't middle-click out of a  
context menu
http://code.google.com/p/chromium/issues/detail?id=5156

Test the bookmarks bar, the context menu works fine (middle-click) and  
events get
passed through. If you notice the difference between that and the other  
ones that
don't work, it is using a different approach to view context menu. It is  
using
chrome_menu.h while the other one is using menu.h.

I believe major refactoring should be done :x Cause if we are going to  
implement this
on the other places, it should be done withing a view, cause bookmarks bar  
overrides
the views OnMousePressed to forward mouse events to the MenuController  
where it deals
with it.

I don't think it is right, to do the same actions everywhere in the code  
that has a
context menu. I think chrome_menu should be revisited to abstract it even  
more so
other places could use it. Can anyone figure out a way to do this? Maybe  
implement
this in the view layer, so all the views will inherit this.

I spent way too much time looking at the code right now :<

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Chromium-bugs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/chromium-bugs?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to