From 35e5e547987b507ed771e751352cba7bd419e41d Mon Sep 17 00:00:00 2001
From: "bzbarsky%mit.edu"
 <bzbarsky%mit.edu@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Tue, 22 Nov 2005 15:51:41 +0000
Subject: [PATCH] Look for the wrapper in the right scope --
 GetWrappedNativeOfNativeObject won't fix up the scope for us.  Bug 309044
 re-fix, r+sr=jst

git-svn-id: svn://10.0.0.236/trunk@185134 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../modules/plugin/base/src/nsJSNPRuntime.cpp | 34 ++++++++++++++++++-
 1 file changed, 33 insertions(+), 1 deletion(-)

diff --git a/mozilla/modules/plugin/base/src/nsJSNPRuntime.cpp b/mozilla/modules/plugin/base/src/nsJSNPRuntime.cpp
index 7dd2983ca2c..ef45a474d2c 100644
--- a/mozilla/modules/plugin/base/src/nsJSNPRuntime.cpp
+++ b/mozilla/modules/plugin/base/src/nsJSNPRuntime.cpp
@@ -48,6 +48,7 @@
 #include "nsIXPConnect.h"
 #include "nsIDOMElement.h"
 #include "prmem.h"
+#include "nsIContent.h"
 
 // Hash of JSObject wrappers that wraps JSObjects as NPObjects. There
 // will be one wrapper per JSObject per plugin instance, i.e. if two
@@ -1409,9 +1410,40 @@ nsJSNPRuntime::OnPluginDestroy(NPP npp)
     return;
   }
 
+  // OK.  Now we have to get our hands on the right scope object, since
+  // GetWrappedNativeOfNativeObject doesn't call PreCreate and hence won't get
+  // the right scope if we pass in something bogus.  The right scope lives on
+  // the script global of the element's document.
+  // XXXbz we MUST have a better way of doing this... perhaps
+  // GetWrappedNativeOfNativeObject _should_ call preCreate?
+  nsCOMPtr<nsIContent> content(do_QueryInterface(element));
+  if (!content) {
+    return;
+  }
+
+  nsIDocument* doc = content->GetOwnerDoc();
+  if (!doc) {
+    return;
+  }
+
+  nsIScriptGlobalObject* sgo = doc->GetScriptGlobalObject();
+  if (!sgo) {
+    return;
+  }
+
+  JSObject* globalObj = sgo->GetGlobalJSObject();
+
+#ifdef DEBUG
+  nsIScriptContext* scx = sgo->GetContext();
+  if (scx) {
+    NS_ASSERTION((JSContext *)scx->GetNativeContext() == cx,
+                 "Unexpected JS context");
+  }
+#endif
+
   nsCOMPtr<nsISupports> supp(do_QueryInterface(element));
   nsCOMPtr<nsIXPConnectWrappedNative> holder;
-  xpc->GetWrappedNativeOfNativeObject(cx, ::JS_GetGlobalObject(cx), supp,
+  xpc->GetWrappedNativeOfNativeObject(cx, globalObj, supp,
                                       NS_GET_IID(nsISupports),
                                       getter_AddRefs(holder));
   if (!holder) {