RESOLVED FIXED 172348
Speedometer 2.0: Vanilla JS test doesn't mark all todo items as completed 
https://bugs.webkit.org/show_bug.cgi?id=172348
Summary Speedometer 2.0: Vanilla JS test doesn't mark all todo items as completed
Ryosuke Niwa
Reported 2017-05-19 00:39:43 PDT
The problem is that the memory storage class uses ms precision wall clock as a storage ID. Because we're adding 50 items at all once, some items would inevitably end up having the same ID. This results in multiple todo items sharing a single ID and cause havoc. Use a monotonically increasing number as an ID instead.
Attachments
Fixes the bug (1.84 KB, patch)
2017-05-19 00:42 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2017-05-19 00:42:48 PDT
Created attachment 310629 [details] Fixes the bug
Antti Koivisto
Comment 2 2017-05-19 01:51:12 PDT
Comment on attachment 310629 [details] Fixes the bug r=me
WebKit Commit Bot
Comment 3 2017-05-19 02:19:05 PDT
Comment on attachment 310629 [details] Fixes the bug Clearing flags on attachment: 310629 Committed r217107: <http://trac.webkit.org/changeset/217107>
WebKit Commit Bot
Comment 4 2017-05-19 02:19:07 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.