Summary: | Regression (r256011): http/tests/resourceLoadStatistics/aggregate-sorted-data-no-storage-access.html is consistently failing | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||||
Component: | Tools / Tests | Assignee: | Kate Cheney <katherine_cheney> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ap, commit-queue, katherine_cheney, tsavell, webkit-bot-watchers-bugzilla, webkit-bug-importer, ysuzuki | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Other | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=207183 | ||||||||
Attachments: |
|
Description
Aakash Jain
2020-02-07 04:42:20 PST
Looks like ResourceLoadStatisticsMemoryStore::dumpResourceLoadStatistics needs to be fixed. The bug is in existing implementation of ResourceLoadStatisticsMemoryStore::dumpResourceLoadStatistics. It relies on particular hash-table ordering, while hash-table does not guarantee it. Created attachment 390102 [details]
Patch
(In reply to Yusuke Suzuki from comment #3) > The bug is in existing implementation of > ResourceLoadStatisticsMemoryStore::dumpResourceLoadStatistics. It relies on > particular hash-table ordering, while hash-table does not guarantee it. Yes, this patch should fix it. I copied what you did in ResourceLoadStatisticsDatabaseStore. Comment on attachment 390102 [details]
Patch
r=me
missed expectations for one test. Uploading a new patch for landing. Created attachment 390115 [details]
Patch for landing
Comment on attachment 390115 [details] Patch for landing Clearing flags on attachment: 390115 Committed r256055: <https://trac.webkit.org/changeset/256055> All reviewed patches have been landed. Closing bug. |