Bug 649599 - Make getAttributeNS return null for absent attributes; r=peterv

content/base/src/nsGenericElement.cpp

@@ -2853,15 +2853,16 @@ nsGenericElement::GetAttributeNS(const nsAString& aNamespaceURI,
     nsContentUtils::NameSpaceManager()->GetNameSpaceID(aNamespaceURI);
 
   if (nsid == kNameSpaceID_Unknown) {
-    // Unknown namespace means no attr...
-
-    aReturn.Truncate();
-
+    // Unknown namespace means no attribute.
+    SetDOMStringToNull(aReturn);
     return NS_OK;
   }
 
   nsCOMPtr<nsIAtom> name = do_GetAtom(aLocalName);
-  GetAttr(nsid, name, aReturn);
+  bool hasAttr = GetAttr(nsid, name, aReturn);
+  if (!hasAttr) {
+    SetDOMStringToNull(aReturn);
+  }
 
   return NS_OK;
 }

dom/tests/mochitest/dom-level2-core/test_getAttributeNS03.html

@@ -105,7 +105,7 @@ function getAttributeNS03() {
       assertNotNull("empAddrNotNull",testAddr);
 testAddr.removeAttributeNS(namespaceURI,localName);
       attrValue = testAddr.getAttributeNS(namespaceURI,localName);
-      assertEquals("throw_Equals","",attrValue);
+      assertEquals("throw_Equals",null,attrValue);
        
 }