From 29568c13e47ed97af936faf82cf00d9149d75bd5 Mon Sep 17 00:00:00 2001 From: "blizzard%redhat.com" Date: Wed, 5 May 2004 19:52:37 +0000 Subject: [PATCH] Bug #242123. Spaces in between the comma and a new-X arguement would not load. Need to verify the URL after arguments have been stripped. r=darin,sr=bryner --- xpfe/components/xremote/src/XRemoteService.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/xpfe/components/xremote/src/XRemoteService.cpp b/xpfe/components/xremote/src/XRemoteService.cpp index b0a62261ca1..3d27e78a39c 100644 --- a/xpfe/components/xremote/src/XRemoteService.cpp +++ b/xpfe/components/xremote/src/XRemoteService.cpp @@ -724,10 +724,6 @@ XRemoteService::OpenURL(nsCString &aArgument, nsIDOMWindowInternal *aParent, PRBool aOpenBrowser) { - // check if we can handle this type of URL - if (!MayOpenURL(aArgument)) - return NS_ERROR_ABORT; - // the eventual toplevel target of the load nsCOMPtr finalWindow = aParent; @@ -768,6 +764,10 @@ XRemoteService::OpenURL(nsCString &aArgument, newWindow = PR_TRUE; } + // check if we can handle this type of URL + if (!MayOpenURL(aArgument)) + return NS_ERROR_ABORT; + // try to fixup the argument passed in nsString url; url.AssignWithConversion(aArgument.get());