tree 2631c624653b66d10d1d80d47cd75f30a8ca0942
parent 1c7e423c7bc33506d9bc8116cb233357af66d0c5
author Sam Rawlins <srawlins@google.com> 1784739696 -0700
committer GitHub <noreply@github.com> 1784739696 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJqYPdwCRC1aQ7uu5UhlAAATU4QADLofMdr7n4MMfHh6JkT2paD
 ddemBST1BSMeZaiXhXYN+xNIM6MCURBQItRFl+PdgteI2BywEzP4AZfi31W74BuA
 oKP1uXWG15WOqRYRQ5LSIt+zMcZWMFtbpbrLqD5Etzfq1VUDZRIvVH5q1aWBP2j5
 igXwlrkRFv6fFHszJvVEkaLQwxS3dWVs/NotSDUNW+EyuCBZmOslko87AeGaoM5v
 LpOo68KfI1eK8Up2xEkb7r9w2caB8L5SvJ69Ol6Iszaq6cEIrjgxSNHLVZR4Wqq5
 kr9mwhAD42bngFSkA01nu5GoRPiFhF2Y/Ii5bPUBkwIa+w9cnwh15gWgF0gD+JeH
 Xx0txCJdLkE3G2dAdCHVsXQoDaHCsJKfIBKxVpL/wTzUoZes8xrPl+OL37IGUYdY
 XZ4mDNqfF1KDDdHSI52Uq91PKdglDvtXM0dHFKHu+JxMKxtBNYkB/VlTkrs6Kyl1
 kW4XGBF3VpBMwj/ETWXqQEYyv8ZgMaMf+iXPClB8khLnBnFsf5Kgek0ocd+uf9gs
 yt1uJhblistog56xncT6H8sFKEGS6N1JcBkjZzahnMwCbuXZlf52fiG7bVxnLnXO
 xbKeBWPHjf0ImVCmcRMGCqnBj32gVE49l9Aai94a/yBipDm0XgVybw6UxJS9YJJ3
 d6HLxQUT25UzIY8x7/b8
 =nY81
 -----END PGP SIGNATURE-----
 

Correct units and cumulative bug in GC times (#9890)

Fixes #7080

In the logging page, GC events were displayed with summaries like idle
collection in 290 ms. The value 290 was calculated directly from the cumulative
times (`newSpace.time` and `oldSpace.time`), meaning it represented the total
cumulative GC time since isolate startup rather than the duration of this
specific GC. In reality, the individual collection took 290 microseconds (which
is 0.29 milliseconds), but since it was displaying the cumulative time of 290
ms, it was off by a factor of 1000x.

In this fix, we update `_handleGCEvent` to track and subtract the previous
cumulative GC time from the new cumulative GC time to compute the actual
duration delta for this specific GC.

Also format the duration as a millisecond double with 1 decimal place
(consistent with how frame times are displayed), and appended it to the log
summary (e.g. "idle collection in 100.0 ms").