From 19a4f15eb67fd82a0b78b12dd94e3564504f83f9 Mon Sep 17 00:00:00 2001 From: Yury Semikhatsky Date: Tue, 9 Jan 2024 13:35:51 -0800 Subject: [PATCH] chore: remove redundant checks from network interception (#28922) We only intercept requests in Playwright so these fields are always undefined. --- .../playwright-core/src/server/chromium/crNetworkManager.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/playwright-core/src/server/chromium/crNetworkManager.ts b/packages/playwright-core/src/server/chromium/crNetworkManager.ts index ac22de3634..0c3411d0d0 100644 --- a/packages/playwright-core/src/server/chromium/crNetworkManager.ts +++ b/packages/playwright-core/src/server/chromium/crNetworkManager.ts @@ -305,7 +305,7 @@ export class CRNetworkManager { }); this._requestIdToRequest.set(requestWillBeSentEvent.requestId, request); - if (requestPausedEvent && !requestPausedEvent.responseStatusCode && !requestPausedEvent.responseErrorReason) { + if (requestPausedEvent) { // We will not receive extra info when intercepting the request. // Use the headers from the Fetch.requestPausedPayload and release the allHeaders() // right away, so that client can call it from the route handler.