[webkit-reviews] review granted: [Bug 73686] IndexedDB: Implement IDBObjectStore.count() and IDBIndex.count() : [Attachment 118600] Patch

[bugzilla-daemon at webkit.org]

Tony Chang <tony at chromium.org> has granted Joshua Bell <jsbell at chromium.org>'s
request for review:
Bug 73686: IndexedDB: Implement IDBObjectStore.count() and IDBIndex.count()
https://bugs.webkit.org/show_bug.cgi?id=73686

Attachment 118600: Patch
https://bugs.webkit.org/attachment.cgi?id=118600&action=review

------- Additional Comments from Tony Chang <tony at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=118600&action=review


> Source/WebCore/storage/IDBObjectStoreBackendImpl.cpp:566
> +    do {
> +	   ++count;
> +    } while (backingStoreCursor->continueFunction(0));

This seems really inefficient.	Is this the only way to get the count?