Shutdown Telemetry IPC accumulator when connection to parent process is closed

NEW
Unassigned

Status

()

defect
P3
normal
3 years ago
2 years ago

People

(Reporter: gfritzsche, Unassigned)

Tracking

(Blocks 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 affected)

Details

(Whiteboard: [measurement:client])

From logging like bug 1339568, comment 1, we still try to send Telemetry messages to the parent after the connection is closed.

I think we should:
- stop exposing IPCTimerFired() in the header
- improve nsITelemetry::FlushBatchChildTelemetry
  - make it replace TelemetryIPCAccumulator::DeInitializeGlobalState
  - make sure we don't batch & send child Telemetry afterwards anymore
You need to log in before you can comment on or make changes to this bug.