[gnome-shell/wip/message-tray-menu: 4/20] messageTray: Use a PopupMenuManager for the message tray context menu

[Jasper St. Pierre <jstpierre src gnome org>]

commit 0c9e43b0fa3911307249c86815b0c70927537876
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Fri Apr 19 11:46:06 2013 -0400

    messageTray: Use a PopupMenuManager for the message tray context menu

 js/ui/messageTray.js |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)
---
diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 2099c90..0e0bf8f 100644
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1720,6 +1720,8 @@ const MessageTray = new Lang.Class({
         this._updateNoMessagesLabel();
 
         this._contextMenu = new MessageTrayContextMenu(this);
+        this._contextMenuManager = new PopupMenu.PopupMenuManager({ grabHelper: this._grabHelper });
+        this._contextMenuManager.addMenu(this._contextMenu);
 
         let clickAction = new Clutter.ClickAction();
         this.actor.add_action(clickAction);
@@ -1728,8 +1730,6 @@ const MessageTray = new Lang.Class({
             let button = action.get_button();
             if (button == 3)
                 this._openContextMenu();
-            if (button == 1 && this._contextMenu.isOpen)
-                this._grabHelper.ungrab({ actor: this._contextMenu.actor });
         }));
 
         clickAction.connect('long-press', Lang.bind(this, function(action, actor, state) {
@@ -1746,12 +1746,6 @@ const MessageTray = new Lang.Class({
     _openContextMenu: function () {
         let [x, y, mask] = global.get_pointer();
         this._contextMenu.setPosition(Math.round(x), Math.round(y));
-        this._grabHelper.grab({ actor: this._contextMenu.actor,
-                                modal: true,
-                                onUngrab: Lang.bind(this, function () {
-                                    this._contextMenu.close(BoxPointer.PopupAnimation.FULL);
-                                })
-        });
         this._contextMenu.open(BoxPointer.PopupAnimation.FULL);
     },