From b1e7b9c070fed534bf9c495767285dd50839a47c Mon Sep 17 00:00:00 2001 From: Adam Raine Date: Mon, 24 Feb 2025 12:43:06 -0800 Subject: [PATCH] tests(dbw): revert unload handler expectations --- .../test-definitions/dobetterweb.js | 23 ------------------- 1 file changed, 23 deletions(-) diff --git a/cli/test/smokehouse/test-definitions/dobetterweb.js b/cli/test/smokehouse/test-definitions/dobetterweb.js index 8dc5a7ccb3ff..a73b2396e9c0 100644 --- a/cli/test/smokehouse/test-definitions/dobetterweb.js +++ b/cli/test/smokehouse/test-definitions/dobetterweb.js @@ -249,13 +249,6 @@ const expectations = { description: 'Failed to load resource: the server responded with a status of 404 (Not Found)', sourceLocation: {url: 'http://localhost:10200/favicon.ico'}, }, - { - // Unload handlers were moved behind a permission policy in M135. - _minChromiumVersion: '135', - source: 'violation', - description: 'Permissions policy violation: unload is not allowed in this document.', - sourceLocation: {url: 'http://localhost:10200/dobetterweb/dbw_tester.html'}, - }, ], }, }, @@ -328,8 +321,6 @@ const expectations = { subItems: undefined, }, { - // Unload handlers were moved behind a permission policy in M135. - _maxChromiumVersion: '134', value: 'UnloadHandler', source: { type: 'source-location', @@ -434,8 +425,6 @@ const expectations = { details: { items: [ { - // Unload handlers were moved behind a permission policy in M135. - _maxChromiumVersion: '134', reason: 'The page has an unload handler in the main frame.', failureType: 'Actionable', subItems: { @@ -444,22 +433,10 @@ const expectations = { }], }, }, - { - // Unload handlers were moved behind a permission policy in M135. - _minChromiumVersion: '135', - reason: 'There were permission requests upon navigating away.', - failureType: 'Pending browser support', - subItems: { - items: [{ - frameUrl: 'http://localhost:10200/dobetterweb/dbw_tester.html', - }], - }, - }, { // This issue only appears in the DevTools runner for some reason. // TODO: Investigate why this doesn't happen on the CLI runner. _runner: 'devtools', - _maxChromiumVersion: '134', reason: 'There were permission requests upon navigating away.', failureType: 'Pending browser support', subItems: {