Bug 356225: Duplicate null check in nsWebBrowserPersist.cpp, patch by Ryan Jones <sciguyryan+bugzilla@gmail.com>, r+sr=bzbarsky

embedding/components/webbrowserpersist/src/nsWebBrowserPersist.cpp

@@ -3930,22 +3930,19 @@ nsWebBrowserPersist::SetDocumentBase(
     // Add the BASE element
     if (!baseElement)
     {
-        if (!baseElement)
-        {
-            nsCOMPtr<nsIDOMNode> newNode;
-            if (xmlDoc)
-            {
-                aDocument->CreateElementNS(
-                    kXHTMLNS,
-                    kBase, getter_AddRefs(baseElement));
-            }
-            else
-            {
-                aDocument->CreateElement(
-                    kBase, getter_AddRefs(baseElement));
-            }
-            headElement->AppendChild(baseElement, getter_AddRefs(newNode));
-        }
+      nsCOMPtr<nsIDOMNode> newNode;
+      if (xmlDoc)
+      {
+          aDocument->CreateElementNS(
+              kXHTMLNS,
+              kBase, getter_AddRefs(baseElement));
+      }
+      else
+      {
+          aDocument->CreateElement(
+              kBase, getter_AddRefs(baseElement));
+      }
+      headElement->AppendChild(baseElement, getter_AddRefs(newNode));
     }
     if (!baseElement)
     {