Auto merge of #8926 - frewsxcv:no-map, r=SimonSapin

Prefer 'if let' over 'map() with side effects'

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8926)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2015-12-11 06:53:36 +05:30
commit 0abddae49d

View file

@ -577,12 +577,20 @@ impl XMLHttpRequestMethods for XMLHttpRequest {
let mut buf = String::new();
buf.push_str(&referer_url.scheme);
buf.push_str("://");
referer_url.serialize_host().map(|ref h| buf.push_str(h));
referer_url.port().as_ref().map(|&p| {
if let Some(ref h) = referer_url.serialize_host() {
buf.push_str(h);
}
if let Some(ref p) = referer_url.port().as_ref() {
buf.push_str(":");
buf.push_str(&p.to_string());
});
referer_url.serialize_path().map(|ref h| buf.push_str(h));
}
if let Some(ref h) = referer_url.serialize_path() {
buf.push_str(h);
}
self.request_headers.borrow_mut().set_raw("Referer".to_owned(), vec![buf.into_bytes()]);
},
Ok(Some(ref req)) => self.insert_trusted_header("origin".to_owned(),