Fixed outlining spans for chained method calls. (#35860)

Author: dragomirtitian

src/services/outliningElementsCollector.ts

@@ -45,7 +45,14 @@ namespace ts.OutliningElementsCollector {
             if (span) out.push(span);
 
             depthRemaining--;
-            if (isIfStatement(n) && n.elseStatement && isIfStatement(n.elseStatement)) {
+            if (isCallExpression(n)) {
+                depthRemaining++;
+                visitNonImportNode(n.expression);
+                depthRemaining--;
+                n.arguments.forEach(visitNonImportNode);
+                n.typeArguments?.forEach(visitNonImportNode);
+            }
+            else if (isIfStatement(n) && n.elseStatement && isIfStatement(n.elseStatement)) {
                 // Consider an 'else if' to be on the same depth as the 'if'.
                 visitNonImportNode(n.expression);
                 visitNonImportNode(n.thenStatement);

tests/cases/fourslash/getOutliningSpansDepthChainedCalls.ts

@@ -0,0 +1,116 @@
+/// <reference path="fourslash.ts"/>
+
+// Tests that each 'else if' does not count towards a higher nesting depth.
+
+////declare var router: any;
+////router
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+////    .get("/", async(ctx) =>[|{
+////        ctx.body = "base";
+////    }|])
+////    .post("/a", async(ctx) =>[|{
+////        //a
+////    }|])
+
+verify.outliningSpansInCurrentFile(test.ranges());