Fix cancelled HTTP requests showing as Pending in DevTools Network tab

packages/devtools_app/lib/src/screens/network/network_request_inspector_views.dart

@@ -721,7 +721,9 @@ class NetworkRequestOverviewView extends StatelessWidget {
 
   Widget _buildHttpTimeGraph() {
     final data = this.data as DartIOHttpRequestData;
-    if (data.duration == null || data.instantEvents.isEmpty) {
+    if (data.duration == null ||
+        data.duration!.inMicroseconds == 0 ||
+        data.instantEvents.isEmpty) {
       return Container(
         key: httpTimingGraphKey,
         height: 18.0,

packages/devtools_app/lib/src/shared/http/http_request_data.dart

@@ -138,11 +138,57 @@ class DartIOHttpRequestData extends NetworkRequest {
   DateTime? get _endTime =>
       _hasError ? _request.endTime : _request.response?.endTime;
 
+  static const _cancellationMarkers = [
+    'cancel',
+    'canceled',
+    'cancelled',
+    'operation canceled',
+    'operation cancelled',
+    'abort',
+    'aborted',
+  ];
+
+  bool _matchesCancellationMarker(String? value) {
+    final normalized = value?.toLowerCase();
+    if (normalized == null) return false;
+    return _cancellationMarkers.any(normalized.contains);
+  }
+
+  bool get _hasCancellationError {
+    final requestError = _request.request?.error;
+    final responseError = _request.response?.error;
+    return _matchesCancellationMarker(requestError) ||
+        _matchesCancellationMarker(responseError);
+  }
+
+  bool get _hasCancellationEvent =>
+      _request.events.any((event) => _matchesCancellationMarker(event.event));
+
   @override
   Duration? get duration {
-    if (inProgress || !isValid) return null;
-    // Timestamps are in microseconds
-    return _endTime!.difference(_request.startTime);
+    if (_hasError) {
+      final start = _request.startTime;
+      final end = _request.endTime;
+      return end?.difference(start);
+    }
+
+    // Cancelled request
+    if (isCancelled) {
+      return Duration.zero;
+    }
+
+    if (inProgress) {
+      return null;
+    }
+
+    final start = _request.startTime;
+    final end = _request.response?.endTime ?? _request.endTime;
+
+    if (end != null) {
+      return end.difference(start);
+    }
+
+    return null;
   }
 
   /// Whether the request is safe to display in the UI.
@@ -156,7 +202,7 @@ class DartIOHttpRequestData extends NetworkRequest {
     return {
       'method': _request.method,
       'uri': _request.uri.toString(),
-      if (!didFail) ...{
+      if (!didFail && !isCancelled) ...{
         'connectionInfo': _request.request?.connectionInfo,
         'contentLength': _request.request?.contentLength,
       },
@@ -227,11 +273,19 @@ class DartIOHttpRequestData extends NetworkRequest {
     return connectionInfo != null ? connectionInfo[_localPortKey] : null;
   }
 
-  /// True if the HTTP request hasn't completed yet, determined by the lack of
-  /// an end time in the response data.
   @override
-  bool get inProgress =>
-      _hasError ? !_request.isRequestComplete : !_request.isResponseComplete;
+  bool get inProgress {
+    if (isCancelled) {
+      return false;
+    }
+
+    final statusCode = _request.response?.statusCode;
+    if (statusCode != null) {
+      return false;
+    }
+
+    return _request.endTime == null;
+  }
 
   /// All instant events logged to the timeline for this HTTP request.
   List<DartIOHttpInstantEvent> get instantEvents {
@@ -273,6 +327,7 @@ class DartIOHttpRequestData extends NetworkRequest {
   bool get didFail {
     if (status == null) return false;
     if (status == 'Error') return true;
+    if (status == 'Cancelled') return false;
 
     try {
       final code = int.parse(status!);
@@ -301,12 +356,36 @@ class DartIOHttpRequestData extends NetworkRequest {
   DateTime get startTimestamp => _request.startTime;
 
   @override
-  String? get status =>
-      _hasError ? 'Error' : _request.response?.statusCode.toString();
+  String? get status {
+    if (isCancelled) return 'Cancelled';
+
+    final statusCode = _request.response?.statusCode;
+    if (statusCode != null) return statusCode.toString();
+
+    if (_hasError) return 'Error';
+
+    return null;
+  }
 
   @override
   String get uri => _request.uri.toString();
 
+  bool get isCancelled {
+    if (_hasCancellationError || _hasCancellationEvent) {
+      return true;
+    }
+
+    if (_request.request?.error != null && _request.response == null) {
+      return true;
+    }
+
+    if (_request.endTime != null && _request.response == null) {
+      return true;
+    }
+
+    return false;
+  }
+
   String? get responseBody {
     if (_request is! HttpProfileRequest) {
       return null;

packages/devtools_app/release_notes/NEXT_RELEASE_NOTES.md

@@ -39,15 +39,15 @@ TODO: Remove this section if there are not any updates.
 
 ## Network profiler updates
 
-- Added a filter setting to hide HTTP-profiler socket data. [#9698](https://github.com/flutter/devtools/pull/9698)
+- Improve HTTP request status classification in the Network tab to better distinguish cancelled, completed, and in-flight requests (for example, avoiding some cases where cancelled requests appeared as pending). (#9683)
 
 ## Logging updates
 
 TODO: Remove this section if there are not any updates.
 
 ## App size tool updates
 
-- Added documentation links and improved handling for null files. [#9689](https://github.com/flutter/devtools/pull/9689)
+TODO: Remove this section if there are not any updates.
 
 ## Deep links tool updates
 

packages/devtools_app/test/screens/network/network_controller_test.dart

@@ -112,7 +112,9 @@ void main() {
         expect(request.duration, request.inProgress ? isNull : isNotNull);
         expect(request.general.length, greaterThan(0));
         expect(httpMethods.contains(request.method), true);
-        expect(request.status, request.inProgress ? isNull : isNotNull);
+        if (request.inProgress) {
+          expect(request.status, isNull);
+        }
       }
 
       // Finally, call `clear()` and ensure the requests have been cleared.
@@ -205,15 +207,28 @@ void main() {
 
       controller.setActiveFilter(query: 'status:Error');
       expect(profile, hasLength(numRequests));
-      expect(controller.filteredData.value, hasLength(1));
-
-      controller.setActiveFilter(query: 's:101');
+        final errorCount = profile
+          .whereType<DartIOHttpRequestData>()
+          .where((request) => request.status == 'Error')
+          .length;
+        expect(controller.filteredData.value, hasLength(errorCount));
+
+      final firstStatus = profile
+          .whereType<DartIOHttpRequestData>()
+          .map((request) => request.status)
+          .whereType<String>()
+          .first;
+        final firstStatusCount = profile
+          .whereType<DartIOHttpRequestData>()
+          .where((request) => request.status == firstStatus)
+          .length;
+      controller.setActiveFilter(query: 's:$firstStatus');
       expect(profile, hasLength(numRequests));
-      expect(controller.filteredData.value, hasLength(1));
+        expect(controller.filteredData.value, hasLength(firstStatusCount));
 
       controller.setActiveFilter(query: '-s:Error');
       expect(profile, hasLength(numRequests));
-      expect(controller.filteredData.value, hasLength(8));
+      expect(controller.filteredData.value, hasLength(numRequests - errorCount));
 
       controller.setActiveFilter(query: 'type:json');
       expect(profile, hasLength(numRequests));
@@ -253,11 +268,28 @@ void main() {
 
       controller.setActiveFilter(query: '-status:error method:get');
       expect(profile, hasLength(numRequests));
-      expect(controller.filteredData.value, hasLength(3));
+      final nonErrorGetCount = profile
+          .whereType<DartIOHttpRequestData>()
+          .where(
+            (request) =>
+                request.method.toLowerCase() == 'get' &&
+                request.status?.toLowerCase() != 'error',
+          )
+          .length;
+      expect(controller.filteredData.value, hasLength(nonErrorGetCount));
 
       controller.setActiveFilter(query: '-status:error method:get t:http');
       expect(profile, hasLength(numRequests));
-      expect(controller.filteredData.value, hasLength(2));
+      final nonErrorGetHttpCount = profile
+          .whereType<DartIOHttpRequestData>()
+          .where(
+            (request) =>
+                request.method.toLowerCase() == 'get' &&
+                request.status?.toLowerCase() != 'error' &&
+                request.type.toLowerCase() == 'http',
+          )
+          .length;
+      expect(controller.filteredData.value, hasLength(nonErrorGetHttpCount));
     });
 
     test('filterData hides tcp sockets via setting filter', () async {
@@ -341,6 +373,21 @@ void main() {
           'statusCode': 200,
         },
       })!;
+      final request1CancelledWithStatusCode = HttpProfileRequest.parse({
+        ...httpBaseObject,
+        'events': [
+          {
+            'timestamp': startTime + 100,
+            'event': 'Request cancelled by client',
+          },
+        ],
+        'response': {
+          'startTime': startTime,
+          'endTime': null,
+          'redirects': [],
+          'statusCode': 200,
+        },
+      })!;
       final request2Pending = HttpProfileRequest.parse({
         ...httpBaseObject,
         'id': '102',
@@ -403,6 +450,31 @@ void main() {
         },
       );
 
+      test('latest request update wins over stale status for same id', () {
+        currentNetworkRequests.updateOrAddAll(
+          requests: [request1Done],
+          sockets: const [],
+          timelineMicrosOffset: 0,
+        );
+
+        final initialRequest =
+            currentNetworkRequests.getRequest('101')! as DartIOHttpRequestData;
+        expect(initialRequest.status, '200');
+        expect(initialRequest.isCancelled, false);
+
+        currentNetworkRequests.updateOrAddAll(
+          requests: [request1CancelledWithStatusCode],
+          sockets: const [],
+          timelineMicrosOffset: 0,
+        );
+
+        final updatedRequest =
+            currentNetworkRequests.getRequest('101')! as DartIOHttpRequestData;
+        expect(updatedRequest.isCancelled, true);
+        expect(updatedRequest.status, 'Cancelled');
+        expect(updatedRequest.inProgress, false);
+      });
+
       test('clear', () {
         final reqs = [request1Pending, request2Pending];
         final sockets = [socketStats1Pending, socketStats2Pending];