From 8d6926d579cb22a82554e12ccf0119918e244534 Mon Sep 17 00:00:00 2001
From: "brendan%mozilla.org" <brendan%mozilla.org>
Date: Sat, 21 May 2005 17:35:16 +0000
Subject: [PATCH] Fix 294957 for good.

---
 js/src/xpconnect/src/XPCNativeWrapper.cpp     | 27 +++++++++++++------
 .../xpconnect/src/xpcwrappednativejsops.cpp   |  2 +-
 2 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/js/src/xpconnect/src/XPCNativeWrapper.cpp b/js/src/xpconnect/src/XPCNativeWrapper.cpp
index c4b7ab1cf37e..8fbb7bc33985 100644
--- a/js/src/xpconnect/src/XPCNativeWrapper.cpp
+++ b/js/src/xpconnect/src/XPCNativeWrapper.cpp
@@ -900,6 +900,10 @@ XPC_NW_Equality(JSContext *cx, JSObject *obj, jsval v, JSBool *bp)
   return JS_TRUE;
 }
 
+extern JSBool JS_DLL_CALLBACK
+XPC_WN_Shared_ToString(JSContext *cx, JSObject *obj,
+                       uintN argc, jsval *argv, jsval *vp);
+
 JS_STATIC_DLL_CALLBACK(JSBool)
 XPC_NW_toString(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
                 jsval *rval)
@@ -922,20 +926,27 @@ XPC_NW_toString(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
     XPCNativeWrapper::GetWrappedNative(cx, obj);
 
   JSObject *wn_obj = wrappedNative->GetFlatJSObject();
-  JSObject *wn_pobj;
-  JSProperty *prop;
+  jsval toStringVal;
 
-  // Check whether any object nearer than the last on the prototype
-  // chain, namely Object.prototype, defines a toString property.
-  if (!OBJ_LOOKUP_PROPERTY(cx, wn_obj, id, &wn_pobj, &prop)) {
+  // Check whether toString has been overridden from its XPCWrappedNative
+  // default native method.
+  if (!OBJ_GET_PROPERTY(cx, wn_obj, id, &toStringVal)) {
     return JS_FALSE;
   }
-  JSBool overridden = prop && ::JS_GetPrototype(cx, wn_pobj) != nsnull;
-  OBJ_DROP_PROPERTY(cx, wn_pobj, prop);
+
+  JSBool overridden = JS_TypeOfValue(cx, toStringVal) != JSTYPE_FUNCTION;
+  if (!overridden) {
+    JSObject *toStringFunObj = JSVAL_TO_OBJECT(toStringVal);
+    JSFunction *toStringFun = (JSFunction*) ::JS_GetPrivate(cx, toStringFunObj);
+
+    overridden =
+      ::JS_GetFunctionNative(cx, toStringFun) != XPC_WN_Shared_ToString;
+  }
 
   JSString* str;
   if (overridden) {
-    // Something overrides Object.prototype.toString -- defer to it.
+    // Something overrides XPCWrappedNative.prototype.toString, we
+    // should defer to it.
 
     str = ::JS_ValueToString(cx, OBJECT_TO_JSVAL(wn_obj));
   } else {
diff --git a/js/src/xpconnect/src/xpcwrappednativejsops.cpp b/js/src/xpconnect/src/xpcwrappednativejsops.cpp
index 01b734ed4337..a8df7b4135b3 100644
--- a/js/src/xpconnect/src/xpcwrappednativejsops.cpp
+++ b/js/src/xpconnect/src/xpcwrappednativejsops.cpp
@@ -118,7 +118,7 @@ ToStringGuts(XPCCallContext& ccx)
 
 /***************************************************************************/
 
-JS_STATIC_DLL_CALLBACK(JSBool)
+JSBool JS_DLL_CALLBACK
 XPC_WN_Shared_ToString(JSContext *cx, JSObject *obj,
                        uintN argc, jsval *argv, jsval *vp)
 {