Add support for variadic JSDoc types.

Author: CyrusNajmabadi

src/compiler/checker.ts

@@ -3710,7 +3710,7 @@ module ts {
                         // NYI:
                         break;
                     case SyntaxKind.JSDocVariadicType:
-                        return getTypeFromJSDocType((<JSDocVariadicType>jsDocType).type);
+                        return getTypeFromJSDocVariadicType(<JSDocVariadicType>jsDocType);
                     case SyntaxKind.JSDocConstructorType:
                         // NYI:
                         break;
@@ -3721,6 +3721,13 @@ module ts {
             }
         }
 
+        function getTypeFromJSDocVariadicType(node: JSDocVariadicType): Type {
+            let type = getTypeFromJSDocType(node.type);
+            if (type) {
+                return createArrayType(type);
+            }
+        }
+
         function getTypeForJSDocTypeReference(node: JSDocTypeReference): Type {
             if (node.name.kind === SyntaxKind.Identifier) {
                 switch ((<Identifier>node.name).text) {

src/compiler/utilities.ts

@@ -820,7 +820,26 @@ module ts {
     }
 
     export function hasRestParameters(s: SignatureDeclaration): boolean {
-        return s.parameters.length > 0 && s.parameters[s.parameters.length - 1].dotDotDotToken !== undefined;
+        let lastParameter = lastOrUndefined(s.parameters);
+        if (!lastParameter) {
+            return false;
+        }
+
+        if (s.parserContextFlags & ParserContextFlags.JavaScriptFile) {
+            if (lastParameter.name.kind === SyntaxKind.Identifier) {
+                let lastParameterName = (<Identifier>lastParameter.name).text;
+
+                let docComment = getJSDocComment(s, getSourceFileOfNode(s));
+                if (docComment) {
+                    let parameter = forEach(docComment.parameters, p => p.name === lastParameterName ? p : undefined);
+                    if (parameter) {
+                        return parameter.type.kind === SyntaxKind.JSDocVariadicType;
+                    }
+                }
+            }
+        }
+
+        return lastParameter.dotDotDotToken !== undefined;
     }
 
     export function isLiteralKind(kind: SyntaxKind): boolean {

tests/cases/fourslash/getJavaScriptCompletions6.ts

@@ -0,0 +1,14 @@
+///<reference path="fourslash.ts" />
+
+// @allowNonTsExtensions: true
+// @Filename: Foo.js
+/////**
+//// * @param {...number} a
+//// */
+////function foo(a) {
+////    a./**/
+////}
+
+debugger;
+goTo.marker();
+verify.completionListContains("concat", /*displayText:*/ undefined, /*documentation*/ undefined, "method");

tests/cases/fourslash/getJavaScriptCompletions7.ts

@@ -0,0 +1,13 @@
+///<reference path="fourslash.ts" />
+
+// @allowNonTsExtensions: true
+// @Filename: Foo.js
+/////**
+//// * @param {...number} a
+//// */
+////function foo(a) {
+////    a[0]./**/
+////}
+
+goTo.marker();
+verify.completionListContains("toExponential", /*displayText:*/ undefined, /*documentation*/ undefined, "method");