refactor(es/minifier): Remove mangle.safari10

[Auspicus]

When the mangle.safari10 option is set to true identifiers are re-used in the try/catch block which results in a parse error.

Description:

Using the mangle.safari10 option causes identifiers to be re-used in a try/catch block.

BREAKING CHANGE:

Related issue (if exists):
#8837

[changeset-bot]

⚠️ No Changeset found

Latest commit: e5b1f32

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[CLAassistant]

All committers have signed the CLA.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[Auspicus]

Can use Default::default() here?

[kdy1]

You can do UPDATE=1 cargo test from ./crates/swc_ecma_minifier to update test snapshot files

[Auspicus]

@kdy1 I think we'll have divergence with the terser output for the broken_safari_catch_scope test if we remove this option. Is that OK?

@kzc has an interesting stance on the safari10 option: terser/terser#117 (comment)

[kdy1]

Yeap, it's fine

[kdy1]

swc-bump:

• swc_ecma_transforms_base
• swc_ecma_minifier
• swc_core

[swc-bot]

Automated review comment generated by auto-rebase script

[kdy1]

Can you run UPDATE=1 cargo test --test projects --test tsc from ./crates/swc?

[swc-bot]

Automated review comment generated by auto-rebase script