1013935 - Add telemetry to track the performance of Login Manager initialization

Status: RESOLVED INACTIVE

(Toolkit :: Password Manager, defect, P5)

Description

[:Paolo Amadini]

From bug 853549 comment 83:

::: browser/base/content/browser.js
@@ +1109,5 @@
> +        Services.logins;
> +      } catch (ex) {
> +        Cu.reportError(ex);
> +      }
> +    }, 3000);

It would be interesting to add some simple telemetry probes to see how effective this is. Ideas:

* What percentage of the time do we successfully do any async init (vs it being interrupted by a sync init)?
* What is the time distribution (in seconds after browser-startup) for when the first password manager call is made?
* When a sync-init occurs, how long does it take? (Ditto for async, just out of curiosity.)

This would help us know if 3 seconds is a good value or not, and if the sync-init is still a significant source of jank or not.

Comment 1

[:Paolo Amadini]

This bug looks easy and valuable enough to be put in the Firefox backlog.

Comment 2

[Josh Matthews [:jdm]]

Can you provide some more information required to complete this task?

Comment 3

[:Paolo Amadini]

Matt, we were thinking of making this a good first bug and therefore add details on how to do it, but I'm not sure anymore whether we still want it or not. Since you're doing Logins work right now, deferring this to you.

Comment 4

[Matthew N. [:MattN]]

We're likely to switch the Rust logins backend before implementing this.