Move dirname collection to input

3 files changed, 16 insertions, 20 deletions

diff --git a/components/script/dom/htmlformelement.rs b/components/script/dom/htmlformelement.rs
index ffd0d111cad..6f4d788fa6b 100644
--- a/components/script/dom/htmlformelement.rs
+++ b/components/script/dom/htmlformelement.rs
@@ -745,8 +745,6 @@ impl HTMLFormElement {
                 .map(|field| (&*field.name, field.replace_value(charset))),
         );
 
-        println!("New URL: {url}", url = &load_data.url);
-
         self.plan_to_navigate(load_data, target);
     }
 
@@ -952,22 +950,7 @@ impl HTMLFormElement {
                 match element {
                     HTMLElementTypeId::HTMLInputElement => {
                         let input = child.downcast::<HTMLInputElement>().unwrap();
-
                         data_set.append(&mut input.form_datums(submitter, encoding));
-
-                        // TODO: probably move to input.form_datums(...) function
-                        // 4.10.18.2 https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#submitting-element-directionality:-the-dirname-attribute
-                        let dirname: DOMString = input.DirName();
-                        let dirname_str: &str = &*dirname;
-                        if !dirname_str.is_empty() {
-                            data_set.push(FormDatum {
-                                ty: input.Type(),
-                                name: DOMString::from_string(dirname_str.to_owned()),
-                                value: FormDatumValue::String(DOMString::from(
-                                    input.directionality("auto"),
-                                )),
-                            });
-                        }
                     },
                     HTMLElementTypeId::HTMLButtonElement => {
                         let button = child.downcast::<HTMLButtonElement>().unwrap();
diff --git a/components/script/dom/htmlinputelement.rs b/components/script/dom/htmlinputelement.rs
index 298af937a32..7141f9a4537 100755
--- a/components/script/dom/htmlinputelement.rs
+++ b/components/script/dom/htmlinputelement.rs
@@ -1473,11 +1473,25 @@ impl HTMLInputElement {
         }
 
         // Step 5.12
-        vec![FormDatum {
+        let mut result = vec![FormDatum {
             ty: ty.clone(),
             name: name,
             value: FormDatumValue::String(self.Value()),
-        }]
+        }];
+
+        // 4.10.18.2
+        // https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#submitting-element-directionality:-the-dirname-attribute
+        let dirname: DOMString = self.DirName();
+        let directionality = DOMString::from(self.directionality("auto"));
+        if !dirname.is_empty() {
+            result.push(FormDatum {
+                ty: ty.clone(),
+                name: dirname.clone(),
+                value: FormDatumValue::String(directionality),
+            });
+        }
+
+        result
     }
 
     // https://html.spec.whatwg.org/multipage/#radio-button-group
diff --git a/components/script/dom/window.rs b/components/script/dom/window.rs
index 724a44e9869..ce290caabf7 100644
--- a/components/script/dom/window.rs
+++ b/components/script/dom/window.rs
@@ -2017,7 +2017,6 @@ impl Window {
             }
             // TODO: step 11, navigationType.
             // Step 12, 13
-            println!("ScriptThread::navigate");
             ScriptThread::navigate(pipeline_id, load_data, replace);
         };
     }