Closed Bug 1810704 Opened 12 days ago Closed 9 days ago

Add telemetry for collecting Session History in Session Store

Categories

(Firefox :: Session Restore, task)

task

Tracking

()

RESOLVED FIXED
111 Branch
Tracking Status
firefox110 --- fixed
firefox111 --- fixed

People

(Reporter: farre, Assigned: farre)

References

(Blocks 1 open bug)

Details

Attachments

(3 files, 1 obsolete file)

Add a FX_SESSION_RESTORE_COLLECT_SESSION_HISTORY_MS probe in SHistoryListener.collect.

Assignee: nobody → afarre
Status: NEW → ASSIGNED
Attached file data-review-session-restore.txt (obsolete) —
Attachment #9312572 - Flags: data-review?(chutten)
Attachment #9312572 - Flags: data-review?(chutten) → data-review?(chumphreys)

Comment on attachment 9312572 [details]
data-review-session-restore.txt

I think FX_SESSION_RESTORE_COLLECT_SESSION_HISTORY_MS would be better represented as technical data (Cat 1) rather than interaction data (Cat 2).

As long as I'm understanding this correctly, it seems like while the user triggers the session restore, the actual measurement is being conducted against the collection process. Is that correct?

Otherwise this looks good and happy to data-review+ once that's been discussed or adjusted.

Flags: needinfo?(afarre)

Sure, I'll adjust.

Flags: needinfo?(afarre)
Attachment #9312572 - Attachment is obsolete: true
Attachment #9312572 - Flags: data-review?(chumphreys)

Switched category to 1, as discussed. Makes sense.

Flags: needinfo?(chumphreys)

It looks like the attached data-review request still notes it as interaction data instead of technical data, can you take a look and see if that's the case on your end?

Got some feedback from other data stewards, I'm going to ignore the attachment and finish the review!

Flags: needinfo?(chumphreys) → needinfo?(afarre)

Comment on attachment 9312771 [details]
data-review-session-restore.txt

Data Review Form

  1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?
    Yes, through the Histograms.json file and the Probe Dictionary.

  2. Is there a control mechanism that allows the user to turn the data collection on and off?
    Yes, via the data-collection opt-out in Firefox settings.

  3. If the request is for permanent data collection, is there someone who will monitor the data over time?
    Yes, this will be a permanent data collection and will be monitored by Andreas Farre (afarre@mozilla.com)

  4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
    Cat 1 (technical data)

  5. Is the data collection request for default-on or default-off?
    Default-on

  6. Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?
    No

  7. Is the data collection covered by the existing Firefox privacy notice?
    Yes

  8. Does the data collection use a third-party collection tool? If yes, escalate to legal.
    No

Result

data-review+

Flags: needinfo?(afarre)
Attachment #9312771 - Flags: data-review+

Yeah, I juggled versions of the file and picked the wrong one. Here's the correct one that reflects the actual change. Thanks.

Pushed by afarre@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e344cb12c391
Collect telemetry for session history collection time. r=peterv
Status: ASSIGNED → RESOLVED
Closed: 9 days ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch

Comment on attachment 9312616 [details]
Bug 1810704 - Collect telemetry for session history collection time. r=peterv!

Beta/Release Uplift Approval Request

  • User impact if declined: We're trying to optimize principal serialization/deserialization in bug Bug 1795312, and we need to be able to tell if it is successful. Without beta uplift we'll know less.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Small change with well understood code paths.
  • String changes made/needed:
  • Is Android affected?: No
Attachment #9312616 - Flags: approval-mozilla-beta?

Comment on attachment 9312616 [details]
Bug 1810704 - Collect telemetry for session history collection time. r=peterv!

Approved for 110 beta 4, thanks.

Attachment #9312616 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.