From d03167d9801e6f082107899968a62b5af3be49fd Mon Sep 17 00:00:00 2001 From: Jeena Lee Date: Wed, 12 Oct 2016 12:46:03 -0700 Subject: Fix Request's Headers to be cloned correctly Previously, when `Clone()` was called, Headers of the template Request did not get cloned properly. This commit fixes that issue, and updates the expected wpt results. --- components/script/dom/request.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'components/script/dom/request.rs') diff --git a/components/script/dom/request.rs b/components/script/dom/request.rs index 4cad8753eeb..3358fc63ef5 100644 --- a/components/script/dom/request.rs +++ b/components/script/dom/request.rs @@ -420,7 +420,7 @@ impl Request { r } - fn clone_from(r: &Request) -> Root { + fn clone_from(r: &Request) -> Fallible> { let req = r.request.borrow(); let url = req.url(); let is_service_worker_global_scope = req.is_service_worker_global_scope; @@ -436,8 +436,9 @@ impl Request { *r_clone.request.borrow_mut() = req.clone(); r_clone.body_used.set(body_used); *r_clone.mime_type.borrow_mut() = mime_type; + try!(r_clone.Headers().fill(Some(HeadersInit::Headers(r.Headers())))); r_clone.Headers().set_guard(headers_guard); - r_clone + Ok(r_clone) } pub fn get_request(&self) -> NetTraitsRequest { @@ -613,7 +614,7 @@ impl RequestMethods for Request { } // Step 2 - Ok(Request::clone_from(self)) + Request::clone_from(self) } #[allow(unrooted_must_root)] -- cgit v1.2.3