Add rule to disallow referencing DOM globals

[ajafff]

It's a common mistake to reference DOM globals declared in lib.dom.d.ts:

function onEvent(e: Event) {
    event.target; // intended to use `e` instead
}

• Identifier is used in the value domain (tsutils.getUsageDomain)
• Identifier is declared in lib.dom.d.ts
• exclude Identifier if type has a construct signature to allow for new Event(...), Event.AT_TARGET and e instanceof Event
  • maybe just exclude all Identifiers starting with an uppercase letter
• exclude certain Identifiers:
  • document, window, navigator
  • alert, confirm, prompt (should be banned by another rule)
  • (set|clear)(Timeout|Interval|Immediate)
  • atob and btoa
  • console
  • (session|local)Storage
  • indexedDB
  • (request|cancel)AnimationFrame
  • URL, CSS
  • print
  • and probably many others once users start complaining
• fixer could simply prepend window., but is this really desired? The code will work the same but the lint failure is gone afterwards ...
• optimizations:
  • only execute this rule if lib.dom.d.ts is included in the program
  • looking up the the SourceFile of every declaration is expensive
    • only consider Symbols with ts.SymbolFlags.Variable or ts.SymbolFlags.Function
    • only consider ambient declarations (declare var and declare function)

Rule name: no-restricted-dom-globals?

Ref: palantir/tslint#3824 which imposes the burden on the user to configure every possible global variable.