From b759d63389de570a2cc8aebf4df4663d2ae2190f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matias=20Niemel=C3=A4?= <matias@yearofmoo.com>
Date: Mon, 18 Mar 2019 13:39:10 -0700
Subject: [PATCH] revert: refactor(ivy): Update query-related comments (#29342)

---
 packages/core/src/linker/query_list.ts   | 21 ++++-----------------
 packages/core/src/render3/query.ts       |  4 +---
 packages/core/test/render3/query_spec.ts |  1 -
 tools/public_api_guard/core/core.d.ts    |  2 +-
 4 files changed, 6 insertions(+), 22 deletions(-)

diff --git a/packages/core/src/linker/query_list.ts b/packages/core/src/linker/query_list.ts
index ae483f8f26..a3aec4360f 100644
--- a/packages/core/src/linker/query_list.ts
+++ b/packages/core/src/linker/query_list.ts
@@ -93,33 +93,20 @@ export class QueryList<T>/* implements Iterable<T> */ {
     return this._results.some(fn);
   }
 
-  /**
-   * Returns a copy of the internal results list as an Array.
-   */
   toArray(): T[] { return this._results.slice(); }
 
   [getSymbolIterator()](): Iterator<T> { return (this._results as any)[getSymbolIterator()](); }
 
   toString(): string { return this._results.toString(); }
 
-  /**
-   * Updates the stored data of the query list, and resets the `dirty` flag to `false`, so that
-   * on change detection, it will not notify of changes to the queries, unless a new change
-   * occurs.
-   *
-   * @param resultsTree The results tree to store
-   */
-  reset(resultsTree: Array<T|any[]>): void {
-    this._results = depthFirstFlatten(resultsTree);
+  reset(res: Array<T|any[]>): void {
+    this._results = flatten(res);
     (this as{dirty: boolean}).dirty = false;
     (this as{length: number}).length = this._results.length;
     (this as{last: T}).last = this._results[this.length - 1];
     (this as{first: T}).first = this._results[0];
   }
 
-  /**
-   * Triggers a change event by emitting on the `changes` {@link EventEmitter}.
-   */
   notifyOnChanges(): void { (this.changes as EventEmitter<any>).emit(this); }
 
   /** internal */
@@ -132,9 +119,9 @@ export class QueryList<T>/* implements Iterable<T> */ {
   }
 }
 
-function depthFirstFlatten<T>(list: Array<T|T[]>): T[] {
+function flatten<T>(list: Array<T|T[]>): T[] {
   return list.reduce((flat: any[], item: T | T[]): T[] => {
-    const flatItem = Array.isArray(item) ? depthFirstFlatten(item) : item;
+    const flatItem = Array.isArray(item) ? flatten(item) : item;
     return (<T[]>flat).concat(flatItem);
   }, []);
 }
diff --git a/packages/core/src/render3/query.ts b/packages/core/src/render3/query.ts
index 741ce9a99d..5ec63a9824 100644
--- a/packages/core/src/render3/query.ts
+++ b/packages/core/src/render3/query.ts
@@ -365,9 +365,7 @@ export function query<T>(
 /**
  * Refreshes a query by combining matches from all active views and removing matches from deleted
  * views.
- *
- * @returns `true` if a query got dirty during change detection or if this is a static query
- * resolving in creation mode, `false` otherwise.
+ * Returns true if a query got dirty during change detection, false otherwise.
  */
 export function queryRefresh(queryList: QueryList<any>): boolean {
   const queryListImpl = (queryList as any as QueryList_<any>);
diff --git a/packages/core/test/render3/query_spec.ts b/packages/core/test/render3/query_spec.ts
index 2221117b52..46c20f5376 100644
--- a/packages/core/test/render3/query_spec.ts
+++ b/packages/core/test/render3/query_spec.ts
@@ -1603,7 +1603,6 @@ describe('query', () => {
          });
 
       // https://stackblitz.com/edit/angular-7vvo9j?file=src%2Fapp%2Fapp.component.ts
-      // https://stackblitz.com/edit/angular-xzwp6n
       it('should report results when the same TemplateRef is inserted into different ViewContainerRefs',
          () => {
            let tpl: TemplateRef<{}>;
diff --git a/tools/public_api_guard/core/core.d.ts b/tools/public_api_guard/core/core.d.ts
index 905f9020d5..329a3a23e5 100644
--- a/tools/public_api_guard/core/core.d.ts
+++ b/tools/public_api_guard/core/core.d.ts
@@ -721,7 +721,7 @@ export declare class QueryList<T> {
     map<U>(fn: (item: T, index: number, array: T[]) => U): U[];
     notifyOnChanges(): void;
     reduce<U>(fn: (prevValue: U, curValue: T, curIndex: number, array: T[]) => U, init: U): U;
-    reset(resultsTree: Array<T | any[]>): void;
+    reset(res: Array<T | any[]>): void;
     setDirty(): void;
     some(fn: (value: T, index: number, array: T[]) => boolean): boolean;
     toArray(): T[];