Closed Bug 1364898 Opened 8 years ago Closed 6 years ago

Generate process enums & strings from Processes.yaml

Categories

(Toolkit :: Telemetry, enhancement, P3)

enhancement
Points:
2

Tracking

()

RESOLVED DUPLICATE of bug 1361661

People

(Reporter: gfritzsche, Unassigned)

References

Details

Attachments

(1 file)

- use Processes.yaml to specify: - the payload processes names - "internal" enum names - (also the valid record_in_processes names?) - generate headers with enums and strings at build time - refactor histograms/scalars/events to use these common headers
Attached patch WIP, incompleteSplinter Review
(In reply to Georg Fritzsche [:gfritzsche] from comment #0) > - use Processes.yaml to specify: > - the payload processes names > - "internal" enum names > - (also the valid record_in_processes names?) > - generate headers with enums and strings at build time > - refactor histograms/scalars/events to use these common headers Also: The C++-to-JS serializations can already use the right payload names (or look them up in a single table), so we could remove the hardcoded name mappings from TelemetrySession.jsm and aboutTelemetry.js.
Depends on: 1328230
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: