From 54d37d920cf37890e5a1615cc531f4762ef8f02b Mon Sep 17 00:00:00 2001 From: Anthony Ramine Date: Sat, 25 Mar 2017 14:43:28 +0100 Subject: Remove some useless Option wrappers from ServoUrl methods --- components/script/dom/htmlformelement.rs | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) (limited to 'components/script/dom/htmlformelement.rs') diff --git a/components/script/dom/htmlformelement.rs b/components/script/dom/htmlformelement.rs index b958e8f1225..afcfd9c6690 100755 --- a/components/script/dom/htmlformelement.rs +++ b/components/script/dom/htmlformelement.rs @@ -384,12 +384,12 @@ impl HTMLFormElement { fn mutate_action_url(&self, form_data: &mut Vec, mut load_data: LoadData, encoding: EncodingRef) { let charset = &*encoding.whatwg_name().unwrap(); - if let Some(ref mut url) = load_data.url.as_mut_url() { - url.query_pairs_mut().clear() - .encoding_override(Some(self.pick_encoding())) - .extend_pairs(form_data.into_iter() - .map(|field| (field.name.clone(), field.replace_value(charset)))); - } + load_data.url + .as_mut_url() + .query_pairs_mut().clear() + .encoding_override(Some(self.pick_encoding())) + .extend_pairs(form_data.into_iter() + .map(|field| (field.name.clone(), field.replace_value(charset)))); self.plan_to_navigate(load_data); } @@ -403,13 +403,12 @@ impl HTMLFormElement { let charset = &*encoding.whatwg_name().unwrap(); load_data.headers.set(ContentType::form_url_encoded()); - - if let Some(ref mut url) = load_data.url.as_mut_url() { - url.query_pairs_mut().clear() - .encoding_override(Some(self.pick_encoding())) - .extend_pairs(form_data.into_iter() - .map(|field| (field.name.clone(), field.replace_value(charset)))); - } + load_data.url + .as_mut_url() + .query_pairs_mut().clear() + .encoding_override(Some(self.pick_encoding())) + .extend_pairs(form_data.into_iter() + .map(|field| (field.name.clone(), field.replace_value(charset)))); load_data.url.query().unwrap_or("").to_string().into_bytes() } -- cgit v1.2.3