Merge pull request #12898 from Microsoft/NavigateToFixForIIFE

NavigateTo is not working correctly for declarations in IIFEs

Author: jramsay

src/services/services.ts

@@ -592,9 +592,8 @@ namespace ts {
                             else {
                                 declarations.push(functionDeclaration);
                             }
-
-                            forEachChild(node, visit);
                         }
+                        forEachChild(node, visit);
                         break;
 
                     case SyntaxKind.ClassDeclaration:

tests/cases/fourslash/navigateToIIFE.ts

@@ -0,0 +1,19 @@
+/// <reference path="fourslash.ts" />
+
+// @Filename: file1.ts
+/////*1*/(function () {
+////    "use strict";
+////    function onResume() {
+////    };
+////} )();
+// @Filename: file2.ts
+/////*2*/class EventManager {
+////    public onResume(name: string) { }
+////}
+////class MyOtherEventManager {
+////    public onResume(name: string) { }
+////}
+verify.navigationItemsListCount(3, "onResume");
+verify.navigationItemsListCount(1, "onResume", undefined, test.marker("1").fileName);
+verify.navigationItemsListContains("onResume", "function", "onResume", "exact", test.marker("1").fileName);
+verify.navigationItemsListCount(2, "onResume", undefined, test.marker("2").fileName);