I'm an idiot. When I checked this in on Friday 5/25 I had the fix commented out

during some back and forth testing.  This was r/sr=bredan and a=asa for drivers@mozilla.org
The bug is 78428.
This commit is contained in:
dbragg%netscape.com 2001-05-30 00:24:00 +00:00
parent 0b11af5980
commit 3cc793f4e8

View File

@ -587,10 +587,10 @@ InstallAddSubcomponent(JSContext *cx, JSObject *obj, uintN argc, jsval *argv, js
folder = (nsInstallFolder*)JS_GetPrivate(cx, jsObj);
//jsrefcount saveDepth;
//saveDepth = JS_SuspendRequest(cx);//Need to suspend use of thread or deadlock occurs
jsrefcount saveDepth;
saveDepth = JS_SuspendRequest(cx);//Need to suspend use of thread or deadlock occurs
rv= nativeThis->AddSubcomponent(b0, b1, b2, folder, b4, flags, &nativeRet);
//JS_ResumeRequest(cx, saveDepth);
JS_ResumeRequest(cx, saveDepth);
if (NS_FAILED(rv))
return JS_FALSE;
@ -627,10 +627,10 @@ InstallAddSubcomponent(JSContext *cx, JSObject *obj, uintN argc, jsval *argv, js
folder = (nsInstallFolder*)JS_GetPrivate(cx, jsObj);
//jsrefcount saveDepth;
//saveDepth = JS_SuspendRequest(cx);//Need to suspend use of thread or deadlock occurs
jsrefcount saveDepth;
saveDepth = JS_SuspendRequest(cx);//Need to suspend use of thread or deadlock occurs
rv = nativeThis->AddSubcomponent(b0, b1, b2, folder, b4, &nativeRet);
//JS_ResumeRequest(cx, saveDepth);
JS_ResumeRequest(cx, saveDepth);
if (NS_FAILED(rv))
return JS_FALSE;