Merge pull request microsoft#8312 from RyanCavanaugh/fix8293

Don't error when 'reserved' names appear in ambient contexts

Author: RyanCavanaugh

src/compiler/binder.ts

@@ -1377,7 +1377,8 @@ namespace ts {
  if (inStrictMode &&
  node.originalKeywordKind >= SyntaxKind.FirstFutureReservedWord &&
  node.originalKeywordKind <= SyntaxKind.LastFutureReservedWord &&
- !isIdentifierName(node)) {
+ !isIdentifierName(node) &&
+ !isInAmbientContext(node)) {
 
  // Report error only if there are no parse errors in file
  if (!file.parseDiagnostics.length) {

tests/baselines/reference/ambientNameRestrictions.js

@@ -0,0 +1,8 @@
+//// [ambientNameRestrictions.ts]
+export declare namespace Foo {
+ export var static: any;
+}
+
+
+//// [ambientNameRestrictions.js]
+"use strict";

tests/baselines/reference/ambientNameRestrictions.symbols

@@ -0,0 +1,8 @@
+=== tests/cases/compiler/ambientNameRestrictions.ts ===
+export declare namespace Foo {
+>Foo : Symbol(Foo, Decl(ambientNameRestrictions.ts, 0, 0))
+
+ export var static: any;
+>static : Symbol(static, Decl(ambientNameRestrictions.ts, 1, 12))
+}
+

tests/baselines/reference/ambientNameRestrictions.types

@@ -0,0 +1,8 @@
+=== tests/cases/compiler/ambientNameRestrictions.ts ===
+export declare namespace Foo {
+>Foo : typeof Foo
+
+ export var static: any;
+>static : any
+}
+

tests/cases/compiler/ambientNameRestrictions.ts

@@ -0,0 +1,3 @@
+export declare namespace Foo {
+ export var static: any;
+}