refactor(xml/parser): Simplify logic for parsing elements (#6557)

crates/swc_html_parser/src/lexer/mod.rs

@@ -1067,16 +1067,16 @@ where
             State::TagOpen => {
                 // Consume the next input character:
                 match self.consume_next_char() {
-                    // U+0021 EXCLAMATION MARK (!)
-                    // Switch to the markup declaration open state.
-                    Some('!') => {
-                        self.state = State::MarkupDeclarationOpen;
-                    }
                     // U+002F SOLIDUS (/)
                     // Switch to the end tag open state.
                     Some('/') => {
                         self.state = State::EndTagOpen;
                     }
+                    // U+0021 EXCLAMATION MARK (!)
+                    // Switch to the markup declaration open state.
+                    Some('!') => {
+                        self.state = State::MarkupDeclarationOpen;
+                    }
                     // ASCII alpha
                     // Create a new start tag token, set its tag name to the empty string.
                     // Reconsume in the tag name state.

crates/swc_xml_ast/src/token.rs

@@ -40,10 +40,6 @@ pub enum Token {
         tag_name: JsWord,
         attributes: Vec<AttributeToken>,
     },
-    ShortTag {
-        tag_name: JsWord,
-        attributes: Vec<AttributeToken>,
-    },
     EmptyTag {
         tag_name: JsWord,
         attributes: Vec<AttributeToken>,

crates/swc_xml_codegen/src/lib.rs

@@ -462,7 +462,6 @@ fn escape_string(value: &str, is_attribute_mode: bool) -> String {
             '&' => {
                 result.push_str("&");
             }
-            '\u{00A0}' => result.push_str(" "),
             '"' if is_attribute_mode => result.push_str("""),
             '<' if !is_attribute_mode => {
                 result.push_str("&lt;");

crates/swc_xml_parser/src/error.rs

@@ -80,6 +80,7 @@ impl Error {
             ErrorKind::MissingWhitespaceBetweenDoctypePublicAndSystemIdentifiers => {
                 "Missing whitespace between doctype public and system identifiers".into()
             }
+            ErrorKind::MissingEndTagName => "Missing end tag name".into(),
             ErrorKind::NestedComment => "Nested comment".into(),
             ErrorKind::DoubleHyphenWithInComment => "Double hyper within comment".into(),
             ErrorKind::NoncharacterInInputStream => "Noncharacter in input stream".into(),
@@ -150,6 +151,7 @@ pub enum ErrorKind {
     MissingWhitespaceAfterDoctypeSystemKeyword,
     MissingWhitespaceBeforeDoctypeName,
     MissingWhitespaceBetweenDoctypePublicAndSystemIdentifiers,
+    MissingEndTagName,
     NestedComment,
     DoubleHyphenWithInComment,
     NoncharacterInInputStream,