navigationBar: Avoid double recursion on special property assignment

Author: Andy Hanson

src/services/navigationBar.ts

@@ -277,7 +277,7 @@ namespace ts.NavigationBar {
                     case SpecialPropertyAssignmentKind.PrototypeProperty:
                     case SpecialPropertyAssignmentKind.Prototype:
                         addNodeWithRecursiveChild(node, (node as BinaryExpression).right);
-                        break;
+                        return;
                     case SpecialPropertyAssignmentKind.ThisProperty:
                     case SpecialPropertyAssignmentKind.Property:
                     case SpecialPropertyAssignmentKind.None:

tests/cases/fourslash/navbarNestedCommonJsExports.ts

@@ -0,0 +1,36 @@
+/// <reference path="fourslash.ts" />
+
+// @allowJs: true
+
+// @Filename: /a.js
+////exports.a = exports.b = exports.c = 0;
+
+//TODO: There should be no childItems of consts
+verify.navigationTree({
+    text: "<global>",
+    kind: "script",
+    childItems: [
+        {
+            text: "a",
+            kind: "const",
+            childItems: [
+                {
+                    text: "b",
+                    kind: "const",
+                    childItems: [{ text: "c", kind: "const" }],
+                },
+            ],
+        },
+    ],
+});
+
+verify.navigationBar([
+    {
+        text: "<global>",
+        kind: "script",
+        childItems: [
+            { text: "a", kind: "const" },
+        ],
+    },
+]);
+