Skip to content

Commit 6fc4fe5

Browse files
pfaffeDevtools-frontend LUCI CQ
authored andcommitted
Do not show request response data for failed requests
Fixed: 449751915 Change-Id: Ia7fcf7c3b5d6baa0b68a46dc150da0d44c113e71 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/7021109 Reviewed-by: Danil Somsikov <dsv@chromium.org> Commit-Queue: Philip Pfaffe <pfaffe@chromium.org>
1 parent c9c5aa7 commit 6fc4fe5

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

front_end/panels/network/RequestResponseView.test.ts

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,4 +127,26 @@ describeWithEnvironment('RequestResponseView', () => {
127127

128128
component.detach();
129129
});
130+
131+
it('shows no response data if the request failed', async () => {
132+
const request = SDK.NetworkRequest.NetworkRequest.create(
133+
'requestId' as Protocol.Network.RequestId, urlString`http://devtools-frontend.test/module.wasm`, urlString``,
134+
null, null, null);
135+
request.setContentDataProvider(
136+
() => Promise.resolve(new TextUtils.ContentData.ContentData(
137+
'AGFzbQEAAAABBQFgAAF/AwIBAAcHAQNiYXIAAAoGAQQAQQILACQEbmFtZQAQD3Nob3ctd2FzbS0yLndhdAEGAQADYmFyAgMBAAA=',
138+
true, 'application/wasm')));
139+
request.mimeType = 'application/wasm';
140+
request.finished = true;
141+
request.failed = true;
142+
143+
const component = new Network.RequestResponseView.RequestResponseView(request);
144+
assert.deepEqual(component.getMimeTypeForDisplay(), 'application/wasm');
145+
renderElementIntoDOM(component);
146+
147+
await component.updateComplete;
148+
assert.strictEqual(
149+
component.contentElement.querySelector<HTMLElement>('devtools-widget')?.innerText,
150+
'Nothing to preview\nThis request has no response data available');
151+
});
130152
});

front_end/panels/network/RequestResponseView.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ export const DEFAULT_VIEW: View = (input: ViewInput, output: ViewOutput, target:
5656
i18nString(UIStrings.failedToLoadResponseData), (input.contentData as {error: string}).error,
5757
element))}></devtools-widget>`;
5858
// clang-format on
59-
} else if (input.request.statusCode === 204) {
59+
} else if (input.request.statusCode === 204 || input.request.failed) {
6060
// clang-format off
6161
widget = html`<devtools-widget
6262
.widgetConfig=${widgetConfig(element => new UI.EmptyWidget.EmptyWidget(

0 commit comments

Comments
 (0)