Merge pull request #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;
+}