| commit | e9960b156682f5d8a23adbe7aafa0745ff665c92 | [log] [tgz] |
|---|---|---|
| author | Wolfgang Beyer <[email protected]> | Wed Sep 06 11:37:48 2023 |
| committer | Devtools-frontend LUCI CQ <devtools-frontend-scoped@luci-project-accounts.iam.gserviceaccount.com> | Wed Sep 06 13:35:49 2023 |
| tree | 95c956daceb3076f33a50a1ef26c5961b41ba2f1 | |
| parent | 7be44e26f5585a83dc26d86a596b0ca12c96f6b6 [diff] [blame] |
Correctly set HTTP status text This is a follow up to [1]. It restores the previous handling of `NetworkRequst.statusText` and moves the functionality introduced in [1] to `NetworkRequest.inferredStatusText()`, which is called when displaying the HTTP status text in the UI. [1] https://crrev.com/c/4729682 Bug: 1470363 Change-Id: I7cf9767a90f2a6fce22d2db27d7e8c02bb8dc2da Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/4843633 Auto-Submit: Wolfgang Beyer <[email protected]> Reviewed-by: Kateryna Prokopenko <[email protected]> Commit-Queue: Wolfgang Beyer <[email protected]>
diff --git a/front_end/panels/network/components/RequestHeadersView.ts b/front_end/panels/network/components/RequestHeadersView.ts index 4b6f7e4..d0b96f6 100644 --- a/front_end/panels/network/components/RequestHeadersView.ts +++ b/front_end/panels/network/components/RequestHeadersView.ts
@@ -404,7 +404,7 @@ statusClasses.push('status-with-comment'); } - const statusText = [this.#request.statusCode, this.#request.statusText, comment].join(' '); + const statusText = [this.#request.statusCode, this.#request.getInferredStatusText(), comment].join(' '); // Disabled until https://crbug.com/1079231 is fixed. // clang-format off