Fixing the toolkit core's ShowWindow methods to not SHOW THE DAMN WINDOW

ON THEIR OWN.  Why are these %^*%^&%&^ methods still being used everywhere?

xpfe/AppCores/src/nsToolkitCore.cpp

@@ -216,9 +216,6 @@ nsToolkitCore::ShowWindow(const nsString& aUrl, nsIDOMWindow* aParent) {
   appShell->CreateTopLevelWindow(parent, urlObj, PR_TRUE, getter_AddRefs(window),
                                nsnull, nsnull, NS_SIZETOCONTENT, NS_SIZETOCONTENT);
 
-  if (window)
-    window->Show(PR_TRUE);
-
   return rv;
 }
 
@@ -341,9 +338,6 @@ nsToolkitCore::ShowWindowWithArgs(const nsString& aUrl,
   appShell->CreateTopLevelWindow(parent, urlObj, PR_TRUE, getter_AddRefs(window),
                                nsnull, cb, NS_SIZETOCONTENT, NS_SIZETOCONTENT);
 
-  if (window)
-    window->Show(PR_TRUE);
-
   return rv;
 }