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 7d8984e3e9
commit 67e54c952f
2 changed files with 1 additions and 8 deletions

View File

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

View File

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