diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-09-16 23:14:14 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-09-16 23:14:14 -0500 |
commit | 6ac9dfdd34299d284995f5e7f957f515f4a2c880 (patch) | |
tree | e726cf51be2641da83ed8f9b33e570b93480b03d | |
parent | 2fb4dd9ec3bb4be7d5ea8404964536c89fc8732b (diff) | |
parent | 3aa2ec653a9adede883c513c6c30a49cac2dfeb2 (diff) | |
download | servo-6ac9dfdd34299d284995f5e7f957f515f4a2c880.tar.gz servo-6ac9dfdd34299d284995f5e7f957f515f4a2c880.zip |
Auto merge of #13304 - frewsxcv:ownership, r=jdm
Don't require `PathBuf` ownership if we don't need it.
<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13304)
<!-- Reviewable:end -->
-rw-r--r-- | components/net/file_loader.rs | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/components/net/file_loader.rs b/components/net/file_loader.rs index b9b4c09fffb..1309a305682 100644 --- a/components/net/file_loader.rs +++ b/components/net/file_loader.rs @@ -14,7 +14,7 @@ use std::borrow::ToOwned; use std::error::Error; use std::fs::File; use std::io::Read; -use std::path::PathBuf; +use std::path::Path; use std::sync::Arc; use url::Url; use util::thread::spawn_named; @@ -68,11 +68,11 @@ fn read_all(reader: &mut File, progress_chan: &ProgressSender, cancel_listener: Ok(LoadResult::Cancelled) } -fn get_progress_chan(load_data: LoadData, file_path: PathBuf, +fn get_progress_chan(load_data: LoadData, file_path: &Path, senders: LoadConsumer, classifier: Arc<MimeClassifier>, buf: &[u8]) -> Result<ProgressSender, ()> { let mut metadata = Metadata::default(load_data.url); - let mime_type = guess_mime_type(file_path.as_path()); + let mime_type = guess_mime_type(file_path); metadata.set_content_type(Some(&mime_type)); return start_sending_sniffed_opt(senders, metadata, classifier, buf, load_data.context); } @@ -105,7 +105,7 @@ pub fn factory(load_data: LoadData, }; if cancel_listener.is_cancelled() { - if let Ok(progress_chan) = get_progress_chan(load_data, file_path, + if let Ok(progress_chan) = get_progress_chan(load_data, &file_path, senders, classifier, &[]) { let _ = progress_chan.send(Done(Err(NetworkError::LoadCancelled))); } @@ -114,7 +114,7 @@ pub fn factory(load_data: LoadData, match read_block(reader) { Ok(ReadStatus::Partial(buf)) => { - let progress_chan = get_progress_chan(load_data, file_path, + let progress_chan = get_progress_chan(load_data, &file_path, senders, classifier, &buf).ok().unwrap(); progress_chan.send(Payload(buf)).unwrap(); let read_result = read_all(reader, &progress_chan, &cancel_listener); @@ -126,7 +126,7 @@ pub fn factory(load_data: LoadData, } } Ok(ReadStatus::EOF) => { - if let Ok(chan) = get_progress_chan(load_data, file_path, + if let Ok(chan) = get_progress_chan(load_data, &file_path, senders, classifier, &[]) { let _ = chan.send(Done(Ok(()))); } |