Auto merge of #22244 - georgeroman:response_statustext_default_change, r=jdm

Change Response's statusText default value from 'Ok' to an empty string

<!-- Please describe your changes on the following line: -->
Closes #22238

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors

<!-- Either: -->
- [X] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

<!-- 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/22244)
<!-- Reviewable:end -->

components/net/fetch/methods.rs

@@ -575,6 +575,8 @@ fn scheme_fetch(
                 .headers
                 .typed_insert(ContentType::from(mime::TEXT_HTML_UTF_8));
             *response.body.lock().unwrap() = ResponseBody::Done(vec![]);
+            response.status = Some((StatusCode::OK, "OK".to_string()));
+            response.raw_status = Some((StatusCode::OK.as_u16(), b"OK".to_vec()));
             response
         },
 
@@ -588,6 +590,8 @@ fn scheme_fetch(
                     Response::new(url, ResourceFetchTiming::new(request.timing_type()));
                 *response.body.lock().unwrap() = ResponseBody::Done(bytes);
                 response.headers.typed_insert(ContentType::from(mime));
+                response.status = Some((StatusCode::OK, "OK".to_string()));
+                response.raw_status = Some((StatusCode::OK.as_u16(), b"OK".to_vec()));
                 response
             },
             Err(_) => {
@@ -689,6 +693,9 @@ fn scheme_fetch(
             };
 
             let mut response = Response::new(url, ResourceFetchTiming::new(request.timing_type()));
+            response.status = Some((StatusCode::OK, "OK".to_string()));
+            response.raw_status = Some((StatusCode::OK.as_u16(), b"OK".to_vec()));
+
             if is_range_request {
                 partial_content(&mut response);
             }

components/net_traits/lib.rs

@@ -534,7 +534,7 @@ impl Metadata {
             charset: None,
             headers: None,
             // https://fetch.spec.whatwg.org/#concept-response-status-message
-            status: Some((200, b"OK".to_vec())),
+            status: Some((200, b"".to_vec())),
             https_state: HttpsState::None,
             referrer: None,
             referrer_policy: None,

components/net_traits/response.rs

@@ -131,8 +131,8 @@ impl Response {
             termination_reason: None,
             url: Some(url),
             url_list: vec![],
-            status: Some((StatusCode::OK, "OK".to_string())),
-            raw_status: Some((200, b"OK".to_vec())),
+            status: Some((StatusCode::OK, "".to_string())),
+            raw_status: Some((200, b"".to_vec())),
             headers: HeaderMap::new(),
             body: Arc::new(Mutex::new(ResponseBody::Empty)),
             cache_state: CacheState::None,

components/script/dom/webidls/Response.webidl

@@ -27,7 +27,7 @@ Response implements Body;
 
 dictionary ResponseInit {
   unsigned short status = 200;
-  ByteString statusText = "OK";
+  ByteString statusText = "";
   HeadersInit headers;
 };
 

tests/wpt/metadata/fetch/api/response/response-clone.html.ini

@@ -44,7 +44,3 @@
 
   [Check response clone use structureClone for teed ReadableStreams (DataViewchunk)]
     expected: FAIL
-
-  [Check Response's clone with default values, without body]
-    expected: FAIL
-

tests/wpt/metadata/fetch/api/response/response-init-001.html.ini

@@ -2,7 +2,3 @@
   type: testharness
   [Check default value for body attribute]
     expected: FAIL
-
-  [Check default value for statusText attribute]
-    expected: FAIL
-