Report nsIRequest::status (nsresult) in the marker (#5375)

Follow-up of [Bug
1936578](https://bugzilla.mozilla.org/show_bug.cgi?id=1936576) on
Bugzilla

Author: theskim

src/components/tooltip/NetworkMarker.js

@@ -464,6 +464,14 @@ export function getNetworkMarkerDetails(
     );
   }
 
+  if (payload.requestStatus) {
+    details.push(
+      <TooltipDetail label="Request Status" key="Network-Request Status">
+        {payload.requestStatus}
+      </TooltipDetail>
+    );
+  }
+
   if (payload.responseStatus) {
     details.push(
       <TooltipDetail label="Response Status Code" key="Network-Response Status">

src/test/components/TooltipMarker.test.js

@@ -848,6 +848,27 @@ describe('TooltipMarker', function () {
     );
   });
 
+  it('renders page information for pages with request status', () => {
+    setupWithPayload(
+      getNetworkMarkers({
+        id: 1235,
+        startTime: 19000,
+        fetchStart: 19200.2,
+        endTime: 20433.8,
+        uri: 'https://example.org/index.html',
+        payload: {
+          cache: 'Hit',
+          pri: 8,
+          count: 47027,
+          contentType: 'text/html',
+          requestStatus: 'NS_OK',
+        },
+      })
+    );
+
+    expect(getValueForProperty('Request Status')).toBe('NS_OK');
+  });
+
   it('renders page information for pages with response status code', () => {
     setupWithPayload(
       getNetworkMarkers({

src/types/markers.js

@@ -539,6 +539,9 @@ export type NetworkPayload = {|
   // or we use 'Unset' if no flag is set.
   classOfService?: string,
 
+  // Used to show the request status (nsresult nsIRequest::status)
+  requestStatus?: string,
+
   // Used to show the HTTP response status code
   responseStatus?: number,