Open Bug 1154500 Opened 10 years ago Updated 2 years ago

TelemetryEnvironment should collect heavy measurements on a helper thread

Categories

(Toolkit :: Telemetry, defect, P4)

defect
Points:
2

Tracking

()

People

(Reporter: vladan, Unassigned)

References

Details

(Keywords: main-thread-io, Whiteboard: [measurement:client])

Currently, TelemetryEnvironment calls into XPCOM methods that perform main-thread I/O to collect configuration info such as the DirectWrite version. This is undesirable since it causes main thread jank. We will likely add additional measurements which require file I/O. Telemetry should be either 1) fetching these measurements itself, off the main thread (using use a PromiseWorker & ctypes) or 2) the underlying code should be refactored to fetch the data off the main thread and return the data asynchronously.
Blocks: 1120356
There's a couple of other options: 3) put the file-reading functionality in C++ and expose it to a PromiserWorker via WebIDL instead of using ctypes, and 4) extend AsyncFetchTelemetryData to return this info. I'll leave the decision up to the implementer.
Blocks: 1122482
No longer blocks: 1120356
Blocks: 1201022
No longer blocks: 1122482
We temporarily removed DWriteVersion in bug 1137404 to fix the perf hit, we want to readd that after fixing this here.
Blocks: 1122482
No longer blocks: 1201022
Priority: -- → P2
Was this a problem because it was happening during startup, or would it be a lesser problem if it was to happen (the same, main thread i/o blocking way) some time after the startup?
We don't want any heavy stuff on the main thread (unless we know the browser is idle -- but we can't rely on that here). We especially don't want heavy stuff on the main thread during startup (i.e. a critical path).
Priority: P2 → P3
Whiteboard: [measurement:client]
Points: --- → 2
Priority: P3 → P4
Hi Georg, I would like to take up this bug. Can yo please provide me some pointers?
Flags: needinfo?(gfritzsche)
I don't think this makes for a good starter bug, lets check for other mentored bugs.
Mentor: vladan.bugzilla
Flags: needinfo?(gfritzsche)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.