Bug 692342 - Remove check for !IsInterfacePointer(), given the code a few lines above. r=mrbkap
authorBobby Holley <bobbyholley@gmail.com>
Fri, 25 Nov 2011 17:09:07 -0800
changeset 81590 a8300adcea7242fe1c652eb9745624e8b435f309
parent 81589 a03dd8e80c34d8ae160128641cb7062d05811857
child 81591 47047b832f461be251f969588e920fb278a3dca4
push idunknown
push userunknown
push dateunknown
reviewersmrbkap
bugs692342
milestone11.0a1
Bug 692342 - Remove check for !IsInterfacePointer(), given the code a few lines above. r=mrbkap
js/xpconnect/src/XPCWrappedNative.cpp
--- a/js/xpconnect/src/XPCWrappedNative.cpp
+++ b/js/xpconnect/src/XPCWrappedNative.cpp
@@ -2740,20 +2740,18 @@ CallMethodHelper::ConvertDependentParam(
 
     jsval src;
 
     if (!GetOutParamSource(i, &src))
         return JS_FALSE;
 
     if (paramInfo.IsOut()) {
         if (datum_type.IsPointer() &&
-            !datum_type.IsInterfacePointer() &&
-            isArray) {
+            isArray)
             dp->SetValNeedsCleanup();
-        }
 
         if (!paramInfo.IsIn())
             return JS_TRUE;
     } else {
         NS_ASSERTION(i < mArgc || paramInfo.IsOptional(),
                      "Expected either enough arguments or an optional argument");
         src = i < mArgc ? mArgv[i] : JSVAL_NULL;