Don't set tags empty just because they have bogus attributes. Bug 262346,

patch by Blake Kaplan <mrbkap@gmail.com>, r+sr=bzbarsky

parser/htmlparser/src/nsHTMLTokenizer.cpp

@@ -659,10 +659,13 @@ nsresult nsHTMLTokenizer::ConsumeAttributes(PRUnichar aChar,
         }
       }
       else { //if(NS_ERROR_HTMLPARSER_BADATTRIBUTE==result){
-        aToken->SetEmpty(PR_TRUE);
         IF_FREE(theToken, mTokenAllocator);
-        if(NS_ERROR_HTMLPARSER_BADATTRIBUTE==result)
+        //Bad attributes are not a reason to set empty.
+        if(NS_ERROR_HTMLPARSER_BADATTRIBUTE==result) {
           result=NS_OK;
+        } else {
+          aToken->SetEmpty(PR_TRUE);
+        }
       }
     }//if