Changes to enable an asynchronous creation of a webshell in response to a

window.open call.

docshell/base/nsWebShell.cpp

@@ -240,6 +240,10 @@ public:
   NS_IMETHOD NewWebShell(PRUint32 aChromeMask,
                          PRBool aVisible,
                          nsIWebShell *&aNewWebShell);
+  NS_IMETHOD CanCreateNewWebShell(PRBool& aResult);
+  NS_IMETHOD SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell);
   NS_IMETHOD FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aResult);
   NS_IMETHOD FocusAvailable(nsIWebShell* aFocusedWebShell, PRBool& aFocusTaken);
 
@@ -1889,6 +1893,27 @@ nsWebShell::NewWebShell(PRUint32 aChromeMask,
   return NS_OK;
 }
 
+NS_IMETHODIMP
+nsWebShell::CanCreateNewWebShell(PRBool& aResult)
+{
+  aResult = PR_TRUE;
+  if (nsnull != mContainer) {
+    return mContainer->CanCreateNewWebShell(aResult);
+  }
+  return NS_OK;
+}
+
+NS_IMETHODIMP
+nsWebShell::SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell)
+{
+  if (nsnull != mContainer) {
+    return mContainer->SetNewWebShellInfo(aName, anURL, aOpenerShell, aChromeMask, aNewShell);
+  }
+  return NS_OK;
+}
+
 NS_IMETHODIMP
 nsWebShell::FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aResult)
 {

webshell/public/nsIWebShell.h

@@ -91,7 +91,15 @@ public:
   NS_IMETHOD NewWebShell(PRUint32 aChromeMask,
                          PRBool aVisible,
                          nsIWebShell *&aNewWebShell) = 0;
-
+  
+  // XXX dave sez: if kipp doesn't think NewWebShell should be part of
+  // this interface, then the next two methods probably shouldn't either.
+  // They should move to whereever NewWebShell moves.
+  NS_IMETHOD CanCreateNewWebShell(PRBool& aResult) = 0;
+  NS_IMETHOD SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell) = 0;
+  
   NS_IMETHOD FindWebShellWithName(const PRUnichar* aName,
                                   nsIWebShell*& aResult) = 0;
 

webshell/src/nsWebShell.cpp

@@ -240,6 +240,10 @@ public:
   NS_IMETHOD NewWebShell(PRUint32 aChromeMask,
                          PRBool aVisible,
                          nsIWebShell *&aNewWebShell);
+  NS_IMETHOD CanCreateNewWebShell(PRBool& aResult);
+  NS_IMETHOD SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell);
   NS_IMETHOD FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aResult);
   NS_IMETHOD FocusAvailable(nsIWebShell* aFocusedWebShell, PRBool& aFocusTaken);
 
@@ -1889,6 +1893,27 @@ nsWebShell::NewWebShell(PRUint32 aChromeMask,
   return NS_OK;
 }
 
+NS_IMETHODIMP
+nsWebShell::CanCreateNewWebShell(PRBool& aResult)
+{
+  aResult = PR_TRUE;
+  if (nsnull != mContainer) {
+    return mContainer->CanCreateNewWebShell(aResult);
+  }
+  return NS_OK;
+}
+
+NS_IMETHODIMP
+nsWebShell::SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell)
+{
+  if (nsnull != mContainer) {
+    return mContainer->SetNewWebShellInfo(aName, anURL, aOpenerShell, aChromeMask, aNewShell);
+  }
+  return NS_OK;
+}
+
 NS_IMETHODIMP
 nsWebShell::FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aResult)
 {

webshell/tests/viewer/nsBrowserWindow.cpp

@@ -1743,6 +1743,21 @@ nsBrowserWindow::NewWebShell(PRUint32 aChromeMask,
   return rv;
 }
 
+NS_IMETHODIMP
+nsBrowserWindow::CanCreateNewWebShell(PRBool& aResult)
+{
+  aResult = PR_TRUE; // We can always make a new web shell synchronously
+  return NS_OK;
+}
+
+NS_IMETHODIMP
+nsBrowserWindow::SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell)
+{
+  return NS_OK; // We don't care about this method, since we can make new web shells immediately.
+}
+
 NS_IMETHODIMP
 nsBrowserWindow::FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aResult)
 {

webshell/tests/viewer/nsBrowserWindow.h

@@ -106,6 +106,10 @@ public:
   NS_IMETHOD NewWebShell(PRUint32 aChromeMask,
                          PRBool aVisible,
                          nsIWebShell *&aNewWebShell);
+  NS_IMETHOD CanCreateNewWebShell(PRBool& aResult);
+  NS_IMETHOD SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell);
   NS_IMETHOD FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aResult);
   NS_IMETHOD FocusAvailable(nsIWebShell* aFocusedWebShell, PRBool& aFocusTaken);
 

webshell/tests/viewer/nsXPBaseWindow.cpp

@@ -561,6 +561,20 @@ NS_IMETHODIMP nsXPBaseWindow::NewWebShell(PRUint32 aChromeMask,
   return rv;
 }
 
+NS_IMETHODIMP
+nsXPBaseWindow::CanCreateNewWebShell(PRBool& aResult)
+{
+  aResult = PR_TRUE; // We can always make a new web shell synchronously
+  return NS_OK;
+}
+
+NS_IMETHODIMP
+nsXPBaseWindow::SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell)
+{
+  return NS_OK; // We don't care about this method, since we can make new web shells immediately.
+}
 
 //----------------------------------------
 

webshell/tests/viewer/nsXPBaseWindow.h

@@ -97,6 +97,10 @@ public:
   NS_IMETHOD NewWebShell(PRUint32 aChromeMask,
                          PRBool aVisible,
                          nsIWebShell *&aNewWebShell);
+  NS_IMETHOD CanCreateNewWebShell(PRBool& aResult);
+  NS_IMETHOD SetNewWebShellInfo(const nsString& aName, const nsString& anURL, 
+                                nsIWebShell* aOpenerShell, PRUint32 aChromeMask,
+                                nsIWebShell** aNewShell);
   NS_IMETHOD FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aResult);
   NS_IMETHOD FocusAvailable(nsIWebShell* aFocusedWebShell, PRBool& aFocusTaken);