tree 93b9a7bcb49051f5ee571aeab85fa0323ee59c28
parent 5d97a5520066f525b590e117f786363028a4e990
author Bo Majewski <majewski@chromium.org> 1687932580 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1687932580 +0000

[Files]: Fix UMAs collection for pure JS approach.

When using makeReadEntriesRecursivelyPromise_ we may call the resolve in
the "error" handling branch. This is due to the fact that
util.readEntriesRecursively does not distinguish between true error
condition and the case when we have enough entries. We detect that
condition by checking if there are some entries collected and the
scanning has not been cancelled. However, that branch was missing a call
to recordInterval. Which means that if we use JS code to search for
files and we find more than maxResults, we were not recording any UMAs.
This CL fixes that situation.

Bug: b:288183953
Change-Id: I51a3d18afe28ec2dceab4863cc1096863c0bd473
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4648733
Commit-Queue: Bo Majewski <majewski@chromium.org>
Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1163399}