__decorate helper should not use `this` when targeting modules

<!-- 🚨 STOP 🚨 𝗦𝗧𝗢𝗣 🚨 𝑺𝑻𝑶𝑷 🚨 Half of all issues filed here are duplicates, answered in the FAQ, or not appropriate for the bug tracker. Even if you think you've found a *bug*, please read the FAQ first, especially the Common "Bugs" That Aren't Bugs section! Please help us by doing the following steps before logging an issue: * Search: https://github.com/Microsoft/TypeScript/search?type=Issues * Read the FAQ: https://github.com/Microsoft/TypeScript/wiki/FAQ Please fill in the *entire* template below. --> <!-- Please try to reproduce the issue with the latest published version. It may have already been fixed. For npm: `typescript@next` This is also the 'Nightly' version in the playground: http://www.typescriptlang.org/play/?ts=Nightly --> **TypeScript Version:** 3.7.x-dev.201xxxxx <!-- Search terms you tried before logging this (so others can find this issue more easily) --> **Search Terms:** __decorate this module **Code** ```ts class Foo { @property() x = 1; } ``` **Expected behavior:** Top-level `this` reference is not emitted. **Actual behavior:** The `__decrate` variable is declared like: ```ts var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) { ``` Since top-level `this` is always `undefined` in modules, `(this && this.__decorate) || ` can be omitted. It's only a few bytes, but the presence of top-level `this` also causes warnings in other downstream tools like Rollup. A few other things could be fixed for an ES6+ helper, btw: - Use `const` instead of `var` (allows for some VM optimizations) - Remove `Reflect.decorate` - Don't use `arguments` **Playground Link:** https://www.typescriptlang.org/play/?ts=Nightly#code/MYGwhgzhAEBiD29oG8BQ1oAEAOAne2AprgC4CeAFAJTQAe0AvNAIwDcqAvkA **Related Issues:** None?