diff --git a/components/net/http_loader.rs b/components/net/http_loader.rs index ae815d09f41..2596abb3353 100644 --- a/components/net/http_loader.rs +++ b/components/net/http_loader.rs @@ -793,7 +793,11 @@ fn try_immutable_origin_to_hyper_origin(url_origin: &ImmutableOrigin) -> Option< match *url_origin { ImmutableOrigin::Opaque(_) => Some(HyperOrigin::NULL), ImmutableOrigin::Tuple(ref scheme, ref host, ref port) => { - HyperOrigin::try_from_parts(&scheme, &host.to_string(), Some(port.clone())).ok() + let port = match (scheme.as_ref(), port) { + ("http", 80) | ("https", 443) => None, + _ => Some(*port), + }; + HyperOrigin::try_from_parts(&scheme, &host.to_string(), port).ok() }, } } @@ -1195,6 +1199,13 @@ fn http_network_fetch( .as_ref() .map(|_| uuid::Uuid::new_v4().to_simple().to_string()); + if log_enabled!(log::Level::Info) { + info!("request for {} ({:?})", url, request.method); + for header in request.headers.iter() { + info!(" - {:?}", header); + } + } + // XHR uses the default destination; other kinds of fetches (which haven't been implemented yet) // do not. Once we support other kinds of fetches we'll need to be more fine grained here // since things like image fetches are classified differently by devtools diff --git a/components/net/tests/http_loader.rs b/components/net/tests/http_loader.rs index 0854844f135..297d6de7e5a 100644 --- a/components/net/tests/http_loader.rs +++ b/components/net/tests/http_loader.rs @@ -1450,13 +1450,8 @@ fn test_origin_set() { ); let origin_url = ServoUrl::parse("http://example.com").unwrap(); - // XXX: Not sure about the Some(80) here. origin_url.origin() returns 80 for the port but origin_url returns None. - origin = Origin::try_from_parts( - origin_url.scheme(), - origin_url.host_str().unwrap(), - Some(80), - ) - .unwrap(); + origin = + Origin::try_from_parts(origin_url.scheme(), origin_url.host_str().unwrap(), None).unwrap(); // Test Origin header is set on Get request with CORS mode let mut request = Request::from_init(RequestInit { url: url.clone(), diff --git a/components/script/dom/htmlformelement.rs b/components/script/dom/htmlformelement.rs index 89e2c9410ef..5469f0b7fc5 100755 --- a/components/script/dom/htmlformelement.rs +++ b/components/script/dom/htmlformelement.rs @@ -459,7 +459,8 @@ impl HTMLFormElement { .headers .typed_insert(ContentType::from(mime::APPLICATION_WWW_FORM_URLENCODED)); - self.set_encoding_override(load_data.url.as_mut_url().query_pairs_mut()) + let mut url = load_data.url.clone(); + self.set_encoding_override(url.as_mut_url().query_pairs_mut()) .clear() .extend_pairs( form_data @@ -467,7 +468,7 @@ impl HTMLFormElement { .map(|field| (field.name.clone(), field.replace_value(charset))), ); - load_data.url.query().unwrap_or("").to_string().into_bytes() + url.query().unwrap_or("").to_string().into_bytes() }, FormEncType::FormDataEncoded => { let mime: Mime = format!("multipart/form-data; boundary={}", boundary) diff --git a/tests/wpt/metadata/MANIFEST.json b/tests/wpt/metadata/MANIFEST.json index f038b1f1412..729a0056ba7 100644 --- a/tests/wpt/metadata/MANIFEST.json +++ b/tests/wpt/metadata/MANIFEST.json @@ -454007,7 +454007,7 @@ "support" ], "common/form-submission.py": [ - "467875453c9dc64aac51add3f4a617d941820972", + "b296ac44c57c658ca9e3189e9f79b0f80677acba", "support" ], "common/get-host-info.sub.js": [ @@ -626623,7 +626623,7 @@ "testharness" ], "html/semantics/forms/form-submission-0/submit-entity-body.html": [ - "8363f39ff32cd165a28086b3c594b55a79ced8e5", + "a52bb0dcd2080eabb44ab1ddca379da6a5e5ea9a", "testharness" ], "html/semantics/forms/form-submission-0/submit-file.sub.html": [ diff --git a/tests/wpt/web-platform-tests/common/form-submission.py b/tests/wpt/web-platform-tests/common/form-submission.py index 467875453c9..b296ac44c57 100644 --- a/tests/wpt/web-platform-tests/common/form-submission.py +++ b/tests/wpt/web-platform-tests/common/form-submission.py @@ -6,5 +6,7 @@ def main(request, response): else: result = request.POST.first('foo') == 'bar' + result = result and request.url_parts.query == 'query=1' + return ([("Content-Type", "text/plain")], "OK" if result else "FAIL") diff --git a/tests/wpt/web-platform-tests/html/semantics/forms/form-submission-0/submit-entity-body.html b/tests/wpt/web-platform-tests/html/semantics/forms/form-submission-0/submit-entity-body.html index 8363f39ff32..a52bb0dcd20 100644 --- a/tests/wpt/web-platform-tests/html/semantics/forms/form-submission-0/submit-entity-body.html +++ b/tests/wpt/web-platform-tests/html/semantics/forms/form-submission-0/submit-entity-body.html @@ -96,7 +96,7 @@ function run_simple_test() { var testframe = document.getElementById("testframe"); var testdocument = testframe.contentWindow.document; testdocument.body.innerHTML = - "
" + + "" + test_obj.input + test_obj.submitelement + "
";