From 36bebb76717eec2fe39ecfa7c9d78407e7118d07 Mon Sep 17 00:00:00 2001
From: "neil%parkwaycc.co.uk"
 <neil%parkwaycc.co.uk@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Thu, 14 Sep 2006 06:10:23 +0000
Subject: [PATCH] Don't dismiss the context menu so eagerly for subframe loads
 b=329521 r+sr=jag

git-svn-id: svn://10.0.0.236/trunk@211527 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/suite/browser/nsBrowserStatusHandler.js | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/mozilla/suite/browser/nsBrowserStatusHandler.js b/mozilla/suite/browser/nsBrowserStatusHandler.js
index 7ade7332e49..54b69ff53b6 100644
--- a/mozilla/suite/browser/nsBrowserStatusHandler.js
+++ b/mozilla/suite/browser/nsBrowserStatusHandler.js
@@ -261,7 +261,21 @@ nsBrowserStatusHandler.prototype =
 
   onLocationChange : function(aWebProgress, aRequest, aLocation)
   {
-    document.getElementById("contentAreaContextMenu").hidePopup();
+    if (gContextMenu) {
+      // Optimise for the common case
+      if (aWebProgress.DOMWindow == content)
+        gContextMenu.menu.hidePopup();
+      else {
+        for (var contextWindow = gContextMenu.target.ownerDocument.defaultView;
+             contextWindow != contextWindow.parent;
+             contextWindow = contextWindow.parent) {
+          if (contextWindow == aWebProgress.DOMWindow) {
+            gContextMenu.menu.hidePopup();
+            break;
+          }
+        }
+      }
+    }
 
     // XXX temporary hack for bug 104532.
     // Depends heavily on setOverLink implementation