Bug 647180 - Exceptions in context-menu context listeners interfere w…

…ith the context menu. r=myk
mozilla · Apr 4, 2011 · 61a2a43 · 61a2a43
1 parent ee2e610
commit 61a2a43
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/packages/addon-kit/lib/context-menu.js b/packages/addon-kit/lib/context-menu.js
@@ -474,9 +474,15 @@ const ContextMenuWorker = Worker.compose({
   // true.  popupNode is the node that was context-clicked.
   isAnyContextCurrent: function CMW_isAnyContextCurrent(popupNode) {
     let listeners = this._port._listeners("context");
-    for (let i = 0; i < listeners.length; i++)
-      if (listeners[i].call(this._port._sandbox, popupNode))
-        return true;
+    for (let i = 0; i < listeners.length; i++) {
+      try {
+        if (listeners[i].call(this._port._sandbox, popupNode))
+          return true;
+      }
+      catch (err) {
+        console.exception(err);
+      }
+    }
     return false;
   },