Update comments according to the spec

Author: longvatrong111

html5ever/src/tree_builder/mod.rs

@@ -1377,7 +1377,7 @@ where
         self.insert_element(Push, ns!(html), name, vec![])
     }
 
-    // https://html.spec.whatwg.org/multipage/parsing.html#insert-an-element-at-the-adjusted-insertion-location
+    /// https://html.spec.whatwg.org/multipage/parsing.html#insert-an-element-at-the-adjusted-insertion-location
     fn insert_foreign_element(
         &self,
         tag: Tag,
@@ -1398,8 +1398,8 @@ where
     }
     //§ END
 
-    // https://html.spec.whatwg.org/multipage/parsing.html#parsing-main-inhead
-    // A start tag whose tag name is "template"
+    /// https://html.spec.whatwg.org/multipage/parsing.html#parsing-main-inhead
+    /// A start tag whose tag name is "template"
     fn should_attach_declarative_shadow(&self, tag: &Tag) -> bool {
         let adjusted_insertion_location = self.appropriate_place_for_insertion(None);
 
@@ -1414,7 +1414,7 @@ where
         // template start tag's shadowrootmode is not in the none state
         let is_shadow_root_mode = tag.attrs.iter().any(|attr| {
             attr.name.local == local_name!("shadowrootmode")
-                && (attr.value.to_string() == *"open" || attr.value.to_string() == *"closed")
+                && (attr.value.as_ref() == "open" || attr.value.as_ref() == "closed")
         });
 
         // Check if intended_parent's document allows declarative shadow roots
@@ -1438,8 +1438,8 @@ where
         is_shadow_root_mode && allow_declarative_shadow_roots && adjusted_current_node_not_topmost
     }
 
-    // https://html.spec.whatwg.org/multipage/parsing.html#parsing-main-inhead
-    // A start tag whose tag name is "template"
+    /// https://html.spec.whatwg.org/multipage/parsing.html#parsing-main-inhead
+    /// A start tag whose tag name is "template"
     fn attach_declarative_shadow(
         &self,
         tag: &Tag,

html5ever/src/tree_builder/rules.rs

@@ -162,18 +162,20 @@ where
                     if (self.should_attach_declarative_shadow(&tag)) {
                         // Attach shadow path
 
-                        // Step 1.
+                        // Step 1. Let declarative shadow host element be adjusted current node.
                         let mut shadow_host = self.open_elems.borrow().last().unwrap().clone();
                         if self.is_fragment() && self.open_elems.borrow().len() == 1 {
                             shadow_host = self.context_elem.borrow().clone().unwrap();
                         }
 
-                        // Step 2.
+                        // Step 2. Let template be the result of insert a foreign element for template start tag, with HTML namespace and true.
                         let template = self.insert_foreign_element(tag.clone(), ns!(html), true);
 
                         // Step 3 - 8.
+                        // Attach a shadow root with declarative shadow host element, mode, clonable, serializable, delegatesFocus, and "named".
                         if self.attach_declarative_shadow(&tag, &shadow_host, &template).is_err() {
-                            // Step 8.1.1.
+                            // Step 8.1.1. Insert an element at the adjusted insertion location with template.
+                            // Pop the current template element created in step 2 first.
                             self.pop();
                             self.insert_element_for(tag);
                         }