Use getAll() when available

Author: schmidt-sebastian

packages/firestore/src/local/simple_db.ts

@@ -646,12 +646,29 @@ export class SimpleDbStore<
     range?: IDBKeyRange
   ): PersistencePromise<ValueType[]> {
     const cursor = this.cursor(this.options(indexOrRange, range));
-    const results: ValueType[] = [];
-    return this.iterateCursor(cursor, (key, value) => {
-      results.push(value);
-    }).next(() => {
-      return results;
-    });
+    // Use `getAll()` if the browser supports IndexedDB v3, as it is roughly
+    // 20% faster. Unfortunately, getAll() does not support custom indices.
+    if (
+      typeof indexOrRange !== 'string' &&
+      typeof this.store.getAll === 'function'
+    ) {
+      const request = this.store.getAll(range || null);
+      return new PersistencePromise((resolve, reject) => {
+        request.onerror = (event: Event) => {
+          reject((event.target as IDBRequest).error!);
+        };
+        request.onsuccess = (event: Event) => {
+          resolve((event.target as IDBRequest).result);
+        };
+      });
+    } else {
+      const results: ValueType[] = [];
+      return this.iterateCursor(cursor, (key, value) => {
+        results.push(value);
+      }).next(() => {
+        return results;
+      });
+    }
   }
 
   deleteAll(): PersistencePromise<void>;

packages/firestore/test/unit/local/simple_db.test.ts

@@ -149,6 +149,20 @@ describe('SimpleDb', () => {
     });
   });
 
+  it('can getAll', async () => {
+    await runTransaction(store => {
+      return store
+        .getAll(42)
+        .next(user => {
+          expect(user).to.equal(null);
+          return store.get(1);
+        })
+        .next(user => {
+          expect(user).to.deep.equal(testData[1]);
+        });
+    });
+  });
+
   it('can put', async () => {
     await runTransaction(store => {
       return store.put(dummyUser);