From b2fa4f59768e3758a2c3678453431224af6dbcd2 Mon Sep 17 00:00:00 2001
From: "hyatt%netscape.com" <hyatt%netscape.com>
Date: Wed, 10 Mar 1999 23:31:26 +0000
Subject: [PATCH] Tweaks related to XUL fragments.

---
 rdf/content/src/nsRDFXULBuilder.cpp     |  2 +-
 rdf/datasource/src/nsXULContentSink.cpp | 78 ++++++++++++-------------
 2 files changed, 38 insertions(+), 42 deletions(-)

diff --git a/rdf/content/src/nsRDFXULBuilder.cpp b/rdf/content/src/nsRDFXULBuilder.cpp
index 973705b6cf9d..cf2fcbd97099 100644
--- a/rdf/content/src/nsRDFXULBuilder.cpp
+++ b/rdf/content/src/nsRDFXULBuilder.cpp
@@ -641,7 +641,7 @@ RDFXULBuilderImpl::CreateContents(nsIContent* aElement)
         }
 
         nsCOMPtr<nsIRDFResource> rdfResource;
-        //xulElement->GetResource(rdfResource);
+        xulElement->GetResource(getter_AddRefs(rdfResource));
         if (!rdfResource) {
             NS_ERROR("The fragment root doesn't have an RDF resource behind it.");
             return rv;
diff --git a/rdf/datasource/src/nsXULContentSink.cpp b/rdf/datasource/src/nsXULContentSink.cpp
index 362bfaf9caed..4c64b95f0d21 100644
--- a/rdf/datasource/src/nsXULContentSink.cpp
+++ b/rdf/datasource/src/nsXULContentSink.cpp
@@ -849,46 +849,42 @@ XULContentSinkImpl::Init(nsIDocument* aDocument, nsIWebShell* aWebShell, nsIRDFD
         return NS_ERROR_NULL_POINTER;
     
     nsCOMPtr<nsIXULChildDocument> childDocument;
-    childDocument = do_QueryInterface(mDocument);
-    if (childDocument != nsnull) {
-        childDocument->GetFragmentRoot(&mFragmentRoot);
-        NS_PRECONDITION(mFragmentRoot, "must have a fragment root to place the fragment properly");
-        if (mFragmentRoot) {
-            // We're totally a subdocument. Find the root document's
-            // data source and make assertions there.
-            
-            // First of all, find the root document.
-            nsIDocument* rootDocument; 
-            nsIDocument* currDocument; 
-            currDocument = aDocument; 
-            NS_ADDREF(currDocument); 
-            while (currDocument != nsnull) { 
-                NS_IF_RELEASE(rootDocument); 
-                rootDocument = currDocument; 
-                currDocument = rootDocument->GetParentDocument(); 
-            } 
+    childDocument = do_QueryInterface(aDocument);
+    childDocument->GetFragmentRoot(&mFragmentRoot);
+    if (mFragmentRoot) {
+        // We're totally a subdocument. Find the root document's
+        // data source and make assertions there.
+      
+        // First of all, find the root document.
+        nsIDocument* rootDocument = nsnull; 
+        nsIDocument* currDocument; 
+        currDocument = aDocument; 
+        NS_ADDREF(currDocument); 
+        while (currDocument != nsnull) { 
+            NS_IF_RELEASE(rootDocument); 
+            rootDocument = currDocument; 
+            currDocument = rootDocument->GetParentDocument(); 
+        } 
 
-            // Retrieve the root data source. 
-            nsCOMPtr<nsIRDFDocument> rdfRootDoc; 
-            rdfRootDoc = do_QueryInterface(rootDocument); 
-            if (rdfRootDoc == nsnull) { 
-                NS_ERROR("Root document of a XUL fragment is not an RDF doc."); 
-                NS_RELEASE(rootDocument); 
-                return rv; 
-            } 
+        // Retrieve the root data source. 
+        nsCOMPtr<nsIRDFDocument> rdfRootDoc; 
+        rdfRootDoc = do_QueryInterface(rootDocument); 
+        if (rdfRootDoc == nsnull) { 
+            NS_ERROR("Root document of a XUL fragment is not an RDF doc."); 
+            NS_RELEASE(rootDocument); 
+            return rv; 
+        } 
 
-            nsCOMPtr<nsIRDFDataSource> docDataSource; 
-            if (NS_FAILED(rv = rdfRootDoc->GetDocumentDataSource(getter_AddRefs(docDataSource)))) { 
-                NS_ERROR("Unable to retrieve an RDF document's data source."); 
-                NS_RELEASE(rootDocument); 
-                return rv; 
-            } 
-  
-            NS_IF_RELEASE(mDataSource);
-            mDataSource = docDataSource.get();
-            NS_ADDREF(mDataSource);
-        }
-        else return NS_ERROR_NULL_POINTER;
+        nsCOMPtr<nsIRDFDataSource> docDataSource; 
+        if (NS_FAILED(rv = rdfRootDoc->GetDocumentDataSource(getter_AddRefs(docDataSource)))) { 
+            NS_ERROR("Unable to retrieve an RDF document's data source."); 
+            NS_RELEASE(rootDocument); 
+            return rv; 
+        } 
+
+        NS_IF_RELEASE(mDataSource);
+        mDataSource = docDataSource.get();
+        NS_ADDREF(mDataSource);
     }
     else {
 
@@ -914,8 +910,9 @@ XULContentSinkImpl::Init(nsIDocument* aDocument, nsIWebShell* aWebShell, nsIRDFD
         return rv;
     }
 
-    if (mFragmentRoot) {
+    if (mFragmentRoot == nsnull) {
         // XUL Namespace isn't registered if we're a root document.
+        // We need to register it.
         rv = mNameSpaceManager->RegisterNameSpace(kXULNameSpaceURI, kNameSpaceID_XUL);
         NS_ASSERTION(NS_SUCCEEDED(rv), "unable to register XUL namespace");
     }
@@ -1133,9 +1130,8 @@ XULContentSinkImpl::OpenTag(const nsIParserNode& aNode)
 
     SplitQualifiedName(aNode.GetText(), nameSpaceID, tag);
 
-    // HTML tags all need to be upper-cased
+    // HTML tags must be lowercase
     if (nameSpaceID == kNameSpaceID_HTML) {
-        tag.ToLowerCase();
         if (tag.Equals("script")) {
             return OpenScript(aNode);
         }