720855 - add telemetry time stamp for "end of session restore"

Status: NEW

(Firefox :: Session Restore, defect)

Description

[:Gavin Sharp [email: [e-mail]]]

See bug 717105 for infra. The idea is to TelemetryTimestamps.add() when session restore has completed restoring the session. Ideally, we would only record this when we're doing a restore-on-startup.

Paul: have any tips for the best way to do that?

Comment 1

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

I would count the number of tabs at startup & then decrement for each in restoreTab & then add the timestamp. I would only do it if we're not restoring on demand (because that might never happen)

Comment 2

[Andres Hernandez [:andreshm]]

Paul, can I take this bug?

Comment 3

[Tim Taubert [:ttaubert] (inactive)]

(In reply to Paul O'Shannessy [:zpao] from comment #1)
> I would only do it if we're not restoring on demand (because that might never happen)

Isn't that the default now? Does it still make sense to implement this measurement that way then if we'll get numbers only from a minority of users?

Comment 4

[:Gavin Sharp [email: [e-mail]]]

Probably not. It might still make sense to track an "end of session restore" number, even if it doesn't include actual loading of all the pages.