fix(router): don't use spread operator to workaround an issue in closure compiler (#22884)

vikerman · IgorMinar · commit e6c731f79113 · 2018-03-20T13:30:37.000-07:00
Closure compiler could not handle the spread operator in this one place. Working around it by removing the use of spread operator. PR Close #22884
diff --git a/packages/router/src/utils/collection.ts b/packages/router/src/utils/collection.ts
@@ -84,7 +84,8 @@ export function waitForMap<A, B>(
     }
   });
 
-  return of (...waitHead, ...waitTail).pipe(concatAll(), lastValue(), map(() => res));
+  // Closure compiler has problem with using spread operator here. So just using Array.concat.
+  return of .apply(null, waitHead.concat(waitTail)).pipe(concatAll(), lastValue(), map(() => res));
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,8 @@ export function waitForMap<A, B>(`
`84`	`84`	`}`
`85`	`85`	`});`
`86`	`86`
`87`		`- return of (...waitHead, ...waitTail).pipe(concatAll(), lastValue(), map(() => res));`
	`87`	`+ // Closure compiler has problem with using spread operator here. So just using Array.concat.`
	`88`	`+ return of .apply(null, waitHead.concat(waitTail)).pipe(concatAll(), lastValue(), map(() => res));`
`88`	`89`	`}`
`89`	`90`
`90`	`91`	`/**`