Update browserslist to latest version (#13717)

* Bump browserslist to latest version

* Remove logic to 'upgrade' older versions of Safari

Author: jamesmockett

dotcom-rendering/package.json

@@ -119,7 +119,7 @@
 		"babel-plugin-polyfill-corejs3": "0.12.0",
 		"babel-plugin-transform-runtime": "6.23.0",
 		"body-parser": "1.20.3",
-		"browserslist": "4.23.0",
+		"browserslist": "4.24.4",
 		"buffer": "6.0.3",
 		"chalk": "4.1.2",
 		"clean-css": "5.3.3",

dotcom-rendering/webpack/browser-targets.js

@@ -31,20 +31,19 @@ const rawTargets = getTargetsFromBrowsersList({ browsers });
  * SWC however will not transpile dynamic imports when there are browser targets
  * that do not support them.
  *
- * ios 10.3.0 and safari 10.1.0 do not support dynamic imports:
+ * iOS 10.3.0 does not support dynamic imports:
  *
  * https://caniuse.com/es6-module-dynamic-import
  *
- * So if unsuported versions are encountered we upgrade to the next versions that
- * do support dynamic imports i.e.:
+ * So if an unsupported version is encountered we upgrade to the next version
+ * that does support dynamic imports i.e.:
  *
- * ios 11 and safari 11.1.0
+ * iOS 11
  *
  * This is safe as browsers without dynamic import support should be
  * covered by the dynamic import polyfill.
  *
- * This logic can be removed once ios 10.3 and safari 10.1 no longer
- * appear in browserslist.
+ * This logic can be removed once iOS 10.3 no longer appears in browserslist.
  *
  * @typedef {Object.<string, string>} Targets
  * @param {Targets} targets
@@ -55,14 +54,8 @@ const upgradeTargets = (targets) => {
 		([browser, version]) => {
 			const versions = version.split('.').map(Number);
 			const major = versions[0] ?? 0;
-			const minor = versions[1] ?? 0;
 			if (browser === 'ios' && major < 11) {
 				return ['ios', '11'];
-			} else if (
-				browser === 'safari' &&
-				(major < 11 || (major === 11 && minor === 0))
-			) {
-				return ['safari', '11.1.0'];
 			}
 			return [browser, version];
 		},

dotcom-rendering/webpack/browser-targets.test.ts

@@ -23,7 +23,7 @@ describe('Browser targets are as expected', () => {
 			firefox: '78.0.0',
 			ios: '11', // upgraded
 			opera: '105.0.0',
-			safari: '11.1.0', // upgraded
+			safari: '11.1.0',
 			samsung: '17.0.0',
 		});
 	});
@@ -36,20 +36,6 @@ describe('Upgrade browser targets', () => {
 		expect(upgradeTargets({ ios: '10' })).toEqual({ ios: '11' });
 		expect(upgradeTargets({ ios: '10.3' })).toEqual({ ios: '11' });
 		expect(upgradeTargets({ ios: '10.9.9' })).toEqual({ ios: '11' });
-		// safari < 11.1
-		expect(upgradeTargets({ safari: '10.11' })).toEqual({
-			safari: '11.1.0',
-		});
-		expect(upgradeTargets({ safari: '11' })).toEqual({ safari: '11.1.0' });
-		expect(upgradeTargets({ safari: '11.0' })).toEqual({
-			safari: '11.1.0',
-		});
-		expect(upgradeTargets({ safari: '11.0.11' })).toEqual({
-			safari: '11.1.0',
-		});
-		expect(upgradeTargets({ safari: '11.0.11' })).toEqual({
-			safari: '11.1.0',
-		});
 	});
 	test('do not modify targets for supported later versions', () => {
 		// ios >= 11
@@ -58,13 +44,5 @@ describe('Upgrade browser targets', () => {
 		expect(upgradeTargets({ ios: '11.0.0' })).toEqual({ ios: '11.0.0' });
 		expect(upgradeTargets({ ios: '12' })).toEqual({ ios: '12' });
 		expect(upgradeTargets({ ios: '14.10.1' })).toEqual({ ios: '14.10.1' });
-		// safari >= 11.1
-		expect(upgradeTargets({ safari: '11.1' })).toEqual({ safari: '11.1' });
-		expect(upgradeTargets({ safari: '11.1.0' })).toEqual({
-			safari: '11.1.0',
-		});
-		expect(upgradeTargets({ safari: '12.10.0' })).toEqual({
-			safari: '12.10.0',
-		});
 	});
 });