Chromium Code Reviews| Index: ui/views/widget/desktop_aura/x11_desktop_handler.cc |
| diff --git a/ui/views/widget/desktop_aura/x11_desktop_handler.cc b/ui/views/widget/desktop_aura/x11_desktop_handler.cc |
| index 3aa874da6234a674f7da598cd6b0605dea1b0910..a2bfb79444b80d12ac07cb95aafeeadff1b9a0e5 100644 |
| --- a/ui/views/widget/desktop_aura/x11_desktop_handler.cc |
| +++ b/ui/views/widget/desktop_aura/x11_desktop_handler.cc |
| @@ -138,6 +138,23 @@ uint32_t X11DesktopHandler::Dispatch(const base::NativeEvent& event) { |
| } |
| break; |
| } |
| + // Menus created by Chrome can be drag and drop targets. Since they are |
| + // direct children of the screen root window and have override_redirect |
| + // we cannot use regular _NET_CLIENT_LIST_STACKING property to find them |
| + // and use a separate cache to keep track of them. |
| + // TODO(varkha): Implement caching of all top level X windows and their |
| + // coordinates and stacking order to eliminate repeated calls to X server |
| + // during mouse movement and drag events. |
|
Elliot Glaysher
2014/03/25 20:11:07
...and shaping.
varkha
2014/03/26 00:15:53
Done.
|
| + case CreateNotify: { |
| + XCreateWindowEvent *xcwe = &event->xcreatewindow; |
| + ui::XMenuList::GetInstance()->MaybeRegisterMenu(xcwe->window); |
| + break; |
| + } |
| + case DestroyNotify: { |
| + XDestroyWindowEvent *xdwe = &event->xdestroywindow; |
| + ui::XMenuList::GetInstance()->MaybeUnregisterMenu(xdwe->window); |
| + break; |
| + } |
| } |
| return POST_DISPATCH_NONE; |