Commit 3077e2ca authored by bzbarsky%mit.edu's avatar bzbarsky%mit.edu
Browse files

Throw from our Init() method if we're already initialized, so we don't add

ourselves to various lists twice.  Bug 282784, r=kaie, sr=jst, a=asa
parent 7b45a8e4
Loading
Loading
Loading
Loading
+12 −1
Original line number Diff line number Diff line
@@ -177,7 +177,18 @@ NS_IMETHODIMP
nsSecureBrowserUIImpl::Init(nsIDOMWindow *window)
{
  PR_LOG(gSecureDocLog, PR_LOG_DEBUG,
         ("SecureUI:%p: Init\n", this));
         ("SecureUI:%p: Init: mWindow: %p, window: %p\n", this, mWindow.get(),
          window));

  if (!window) {
    NS_WARNING("Null window passed to nsSecureBrowserUIImpl::Init()");
    return NS_ERROR_INVALID_ARG;
  }

  if (mWindow) {
    NS_WARNING("Trying to init an nsSecureBrowserUIImpl twice");
    return NS_ERROR_ALREADY_INITIALIZED;
  }
  
  nsresult rv = NS_OK;
  mWindow = window;