Skip to content
Snippets Groups Projects
Commit c62ee846 authored by Gerald Squelart's avatar Gerald Squelart
Browse files

Bug 1598992 - More precise way to decide when to delete expired Base Profiler data - r=gregtatum

In practice the previous test that was deleting Base Profiler data when the
index became greater that 1 was correct for Firefox, because the Base Profiler
*always* starts before the very first Gecko Profiler active instance.

However in tests (like the one in the following patch) this may not be true,
because each test may start and stop the profiler, and the recent storage update
means that the index doesn't go back to 1. So when a test (apart from the first
test to use the profiler) attemps to use the Base Profiler, that Base Profiler
data will be immediately discarded because the index is already greater than 1
(from previous tests).

This change is more future-proof as well, in case we later want to use the Base
Profiler more than once in a Firefox instance.

Differential Revision: https://phabricator.services.mozilla.com/D54445

--HG--
extra : moz-landing-system : lando
parent 9da1d33d
No related branches found
No related tags found
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment