mirror of
https://github.com/servo/servo.git
synced 2025-08-06 06:00:15 +01:00
Remove some useless Option<T> wrappers from ServoUrl methods
This commit is contained in:
parent
bc5c4fa892
commit
54d37d920c
6 changed files with 34 additions and 56 deletions
|
@ -384,12 +384,12 @@ impl HTMLFormElement {
|
|||
fn mutate_action_url(&self, form_data: &mut Vec<FormDatum>, 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()
|
||||
}
|
||||
|
|
|
@ -52,9 +52,8 @@ impl URL {
|
|||
}
|
||||
|
||||
pub fn set_query_pairs(&self, pairs: &[(String, String)]) {
|
||||
if let Some(ref mut url) = self.url.borrow_mut().as_mut_url() {
|
||||
url.query_pairs_mut().clear().extend_pairs(pairs);
|
||||
}
|
||||
let mut url = self.url.borrow_mut();
|
||||
url.as_mut_url().query_pairs_mut().clear().extend_pairs(pairs);
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -45,49 +45,31 @@ impl UrlHelper {
|
|||
USVString(quirks::username(url.as_url()).to_owned())
|
||||
}
|
||||
pub fn SetHash(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
quirks::set_hash(url, &value.0)
|
||||
}
|
||||
quirks::set_hash(url.as_mut_url(), &value.0)
|
||||
}
|
||||
pub fn SetHost(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
let _ = quirks::set_host(url, &value.0);
|
||||
}
|
||||
let _ = quirks::set_host(url.as_mut_url(), &value.0);
|
||||
}
|
||||
pub fn SetPort(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
let _ = quirks::set_port(url, &value.0);
|
||||
}
|
||||
let _ = quirks::set_port(url.as_mut_url(), &value.0);
|
||||
}
|
||||
pub fn SetSearch(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
quirks::set_search(url, &value.0)
|
||||
}
|
||||
quirks::set_search(url.as_mut_url(), &value.0)
|
||||
}
|
||||
pub fn SetPathname(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
quirks::set_pathname(url, &value.0)
|
||||
}
|
||||
quirks::set_pathname(url.as_mut_url(), &value.0)
|
||||
}
|
||||
pub fn SetHostname(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
let _ = quirks::set_hostname(url, &value.0);
|
||||
}
|
||||
let _ = quirks::set_hostname(url.as_mut_url(), &value.0);
|
||||
}
|
||||
pub fn SetPassword(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
let _ = quirks::set_password(url, &value.0);
|
||||
}
|
||||
let _ = quirks::set_password(url.as_mut_url(), &value.0);
|
||||
}
|
||||
pub fn SetProtocol(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
let _ = quirks::set_protocol(url, &value.0);
|
||||
}
|
||||
let _ = quirks::set_protocol(url.as_mut_url(), &value.0);
|
||||
}
|
||||
pub fn SetUsername(url: &mut ServoUrl, value: USVString) {
|
||||
if let Some(ref mut url) = url.as_mut_url() {
|
||||
let _ = quirks::set_username(url, &value.0);
|
||||
}
|
||||
let _ = quirks::set_username(url.as_mut_url(), &value.0);
|
||||
}
|
||||
// https://w3c.github.io/webappsec-secure-contexts/#is-origin-trustworthy
|
||||
pub fn is_origin_trustworthy(url: &ServoUrl) -> bool {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue