Add a PerfTest targeting IDBObjectStore.get()
Created attachment 297033 [details] Patch
Comment on attachment 297033 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=297033&action=review > PerformanceTests/IndexedDB/objectstore-get.html:41 > +function *runIteration() I'm surprised that star in front of runIterations does not cause a syntax error.
(In reply to comment #2) > Comment on attachment 297033 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=297033&action=review > > > PerformanceTests/IndexedDB/objectstore-get.html:41 > > +function *runIteration() > > I'm surprised that star in front of runIterations does not cause a syntax > error. Disregard this remark. function* is the syntax to define a generator,<https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/function*>
Comment on attachment 297033 [details] Patch Clearing flags on attachment: 297033 Committed r209772: <http://trac.webkit.org/changeset/209772>
All reviewed patches have been landed. Closing bug.
(In reply to comment #3) > (In reply to comment #2) > > Comment on attachment 297033 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=297033&action=review > > > > > PerformanceTests/IndexedDB/objectstore-get.html:41 > > > +function *runIteration() > > > > I'm surprised that star in front of runIterations does not cause a syntax > > error. > > Disregard this remark. function* is the syntax to define a > generator,<https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/ > Statements/function*> Yup - Because of how asynchronous IndexedDB is, we've fallen into the pattern of function generators to right linear tests for it. :)
(In reply to comment #6) > (In reply to comment #3) > > (In reply to comment #2) > > > Comment on attachment 297033 [details] > > > Patch > > > > > > View in context: > > > https://bugs.webkit.org/attachment.cgi?id=297033&action=review > > > > > > > PerformanceTests/IndexedDB/objectstore-get.html:41 > > > > +function *runIteration() > > > > > > I'm surprised that star in front of runIterations does not cause a syntax > > > error. > > > > Disregard this remark. function* is the syntax to define a > > generator,<https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/ > > Statements/function*> > > Yup - Because of how asynchronous IndexedDB is, we've fallen into the > pattern of function generators to right linear tests for it. :) Wow. To say that again: Yup - Because of how asynchronous IndexedDB is, we've fallen into the pattern of *using* function generators to *write* linear tests for it.