inspector: support Network.Response.statusText property

Author: cola119

lib/internal/inspector_network_tracking.js

@@ -40,6 +40,7 @@ function onClientResponseFinish({ request, response }) {
     response: {
       url,
       status: response.statusCode,
+      statusText: response.statusMessage ?? '',
       headers: response.headers,
     },
   });

src/inspector/network_agent.cc

@@ -17,10 +17,14 @@ std::unique_ptr<Network::Request> createRequest(
 }
 
 std::unique_ptr<Network::Response> createResponse(
-    const String& url, int status, std::unique_ptr<Network::Headers> headers) {
+    const String& url,
+    int status,
+    const String& statusText,
+    std::unique_ptr<Network::Headers> headers) {
   return Network::Response::create()
       .setUrl(url)
       .setStatus(status)
+      .setStatusText(statusText)
       .setHeaders(std::move(headers))
       .build();
 }
@@ -96,6 +100,8 @@ void NetworkAgent::responseReceived(
   response->getString("url", &url);
   int status;
   response->getInteger("status", &status);
+  String statusText;
+  response->getString("statusText", &statusText);
 
   ErrorSupport errors;
   auto headers =
@@ -104,10 +110,11 @@ void NetworkAgent::responseReceived(
     headers = std::make_unique<Network::Headers>(DictionaryValue::create());
   }
 
-  frontend_->responseReceived(request_id,
-                              timestamp,
-                              type,
-                              createResponse(url, status, std::move(headers)));
+  frontend_->responseReceived(
+      request_id,
+      timestamp,
+      type,
+      createResponse(url, status, statusText, std::move(headers)));
 }
 
 void NetworkAgent::loadingFinished(

src/inspector/node_protocol.pdl

@@ -144,6 +144,7 @@ experimental domain Network
     properties
       string url
       integer status
+      string statusText
       Headers headers
 
   # Request / response headers as keys / values of JSON object.

test/parallel/test-inspector-emit-protocol-event.js

@@ -42,6 +42,17 @@ const EXPECTED_EVENTS = {
           status: 200,
           headers: { host: 'nodejs.org' }
         }
+      },
+      expected: {
+        requestId: 'request-id-1',
+        timestamp: 1000,
+        type: 'Other',
+        response: {
+          url: 'https://nodejs.org/en',
+          status: 200,
+          statusText: '', // Status text should be an empty string if not provided.
+          headers: { host: 'nodejs.org' }
+        }
       }
     },
     {

test/parallel/test-inspector-network-domain.js

@@ -61,6 +61,7 @@ const testHttpGet = () => new Promise((resolve, reject) => {
     assert.strictEqual(typeof params.timestamp, 'number');
     assert.strictEqual(params.type, 'Other');
     assert.strictEqual(params.response.status, 200);
+    assert.strictEqual(params.response.statusText, 'OK');
     assert.strictEqual(params.response.url, 'http://127.0.0.1/hello-world');
     assert.strictEqual(typeof params.response.headers, 'object');
   }));
@@ -91,6 +92,7 @@ const testHttpsGet = () => new Promise((resolve, reject) => {
     assert.strictEqual(typeof params.timestamp, 'number');
     assert.strictEqual(params.type, 'Other');
     assert.strictEqual(params.response.status, 200);
+    assert.strictEqual(params.response.statusText, 'OK');
     assert.strictEqual(params.response.url, 'https://127.0.0.1/hello-world');
     assert.strictEqual(typeof params.response.headers, 'object');
   }));