From e60cb419526c293551bae39a660ba6bf804e97f4 Mon Sep 17 00:00:00 2001 From: Bobby Holley Date: Tue, 5 Mar 2013 10:01:59 -0800 Subject: [PATCH] Bug 790732 - Stop attaching Components in InitClasses. r=mrbkap This method is larely deprecated. The only two consumers are JSD and the setup for the safe JSContext, neither of which use system principal and thus neither of which should have |Components|. --- js/xpconnect/src/nsXPConnect.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/js/xpconnect/src/nsXPConnect.cpp b/js/xpconnect/src/nsXPConnect.cpp index 599fd1ff315..47c3d1cde1a 100644 --- a/js/xpconnect/src/nsXPConnect.cpp +++ b/js/xpconnect/src/nsXPConnect.cpp @@ -946,9 +946,6 @@ nsXPConnect::InitClasses(JSContext * aJSContext, JSObject * aGlobalJSObj) scope->RemoveWrappedNativeProtos(); - if (!nsXPCComponents::AttachComponentsObject(ccx, scope)) - return UnexpectedFailure(NS_ERROR_FAILURE); - if (!XPCNativeWrapper::AttachNewConstructorObject(ccx, aGlobalJSObj)) return UnexpectedFailure(NS_ERROR_FAILURE);