Bug 860941 - Remove mArgumentsLast. r=jst

I'm not sure what it used to do, but it sure doesn't do a damn thing now.
This commit is contained in:
Bobby Holley 2013-05-17 10:43:18 -07:00
parent c1e03641e2
commit fcd577710a
2 changed files with 1 additions and 8 deletions

View File

@ -1395,7 +1395,6 @@ nsGlobalWindow::CleanUp(bool aIgnoreModalDialog)
mInnerWindowHolder = nullptr;
mArguments = nullptr;
mArgumentsLast = nullptr;
mArgumentsOrigin = nullptr;
CleanupCachedXBLHandlers(this);
@ -1624,7 +1623,6 @@ NS_IMPL_CYCLE_COLLECTION_TRAVERSE_BEGIN_INTERNAL(nsGlobalWindow)
NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mControllers)
NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mArguments)
NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mArgumentsLast)
NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mPerformance)
@ -1671,7 +1669,6 @@ NS_IMPL_CYCLE_COLLECTION_UNLINK_BEGIN(nsGlobalWindow)
NS_IMPL_CYCLE_COLLECTION_UNLINK(mControllers)
NS_IMPL_CYCLE_COLLECTION_UNLINK(mArguments)
NS_IMPL_CYCLE_COLLECTION_UNLINK(mArgumentsLast)
NS_IMPL_CYCLE_COLLECTION_UNLINK(mPerformance)
@ -2738,7 +2735,6 @@ nsGlobalWindow::DetachFromDocShell()
// We got no new document after someone called
// SetArguments(), drop our reference to the arguments.
mArguments = nullptr;
mArgumentsLast = nullptr;
mArgumentsOrigin = nullptr;
}
@ -3155,9 +3151,7 @@ nsGlobalWindow::SetArguments(nsIArray *aArguments, nsIPrincipal *aOrigin)
nsGlobalWindow *currentInner = GetCurrentInnerWindowInternal();
if (!mIsModalContentWindow) {
mArgumentsLast = aArguments;
} else if (currentInner) {
if (mIsModalContentWindow && currentInner) {
// SetArguments() is being called on a modal content window that
// already has an inner window. This can happen when loading
// javascript: URIs as modal content dialogs. In this case, we'll

View File

@ -1114,7 +1114,6 @@ protected:
nsWeakPtr mOpener;
nsCOMPtr<nsIControllers> mControllers;
nsCOMPtr<nsIArray> mArguments;
nsCOMPtr<nsIArray> mArgumentsLast;
nsCOMPtr<nsIPrincipal> mArgumentsOrigin;
nsRefPtr<Navigator> mNavigator;
nsRefPtr<nsScreen> mScreen;