[webkit-changes] [WebKit/WebKit] 3144f3: PerformanceTests/IndexedDB/stress tests use one da...
Sihui
noreply at github.com
Fri Oct 20 13:22:24 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 3144f3ca572e6621959e68562e2569ff3cbd3ded
https://github.com/WebKit/WebKit/commit/3144f3ca572e6621959e68562e2569ff3cbd3ded
Author: Sihui Liu <sihui_liu at apple.com>
Date: 2023-10-20 (Fri, 20 Oct 2023)
Changed paths:
M PerformanceTests/IndexedDB/stress/large-array-keys.html
M PerformanceTests/IndexedDB/stress/large-binary-keys.html
M PerformanceTests/IndexedDB/stress/large-number-of-inserts-responsiveness.html
M PerformanceTests/IndexedDB/stress/large-number-of-inserts.html
M PerformanceTests/IndexedDB/stress/large-string-keys.html
Log Message:
-----------
PerformanceTests/IndexedDB/stress tests use one database for all iterations unexpectedly
https://bugs.webkit.org/show_bug.cgi?id=263437
rdar://117257501
Reviewed by Youenn Fablet and Brady Eidson.
The done function passed to PerfTestRunner is invoked at the end of all test iterations, not at the end of each
iteration. That means, currentIteration is not updated between iterations and all iterations use the same database (as
database name is created based on currentIteration value). This can lead to misleading results. For example, only the
first iteration spends time in opening the database and the other iterations don't; only the first iteration is adding
items to database, and the other iterations are updating items (since we use put() function in tests).
To fix this, ensure the clean up tasks happen after each test iteration, and tests use different database for different
iterations.
* PerformanceTests/IndexedDB/stress/large-array-keys.html:
* PerformanceTests/IndexedDB/stress/large-binary-keys.html:
* PerformanceTests/IndexedDB/stress/large-number-of-inserts-responsiveness.html:
* PerformanceTests/IndexedDB/stress/large-number-of-inserts.html:
* PerformanceTests/IndexedDB/stress/large-string-keys.html:
Canonical link: https://commits.webkit.org/269584@main
More information about the webkit-changes
mailing list