Don't load TelemetryStartup.js in the content process

RESOLVED FIXED in Firefox 55

Status

()

defect
P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mccr8, Assigned: mccr8)

Tracking

(Blocks 2 bugs)

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [measurement:client])

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
TelemetryStartup.js is an XPCOM component. It gets loaded in every parent and content process, and persists through the lifetime of the process. What it seems to do is listen for profile-after-change (in the parent) or app-startup (in content processes) and runs some code. Maybe this could be turned into a script loaded from MainProcessSingleton.js and ContentProcessSingleton.js.
(Assignee)

Updated

2 years ago
Assignee: nobody → continuation
(Assignee)

Comment 1

2 years ago
The code that TelemetryStartup runs in the content process is really simple, and that's what I care more about anyways (due to the increasing number of content processes) so I'll just fix that here.
Summary: Get rid of TelemetryStartup.js → Don't load TelemetryStartup.js in the content process
Priority: -- → P2
Whiteboard: [measurement:client]
Comment hidden (mozreview-request)

Comment 3

2 years ago
mozreview-review
Comment on attachment 8857517 [details]
Bug 1355611 - Don't load TelemetryStartup.js in the content process.

https://reviewboard.mozilla.org/r/129486/#review132042
Attachment #8857517 - Flags: review?(gfritzsche) → review+
(Assignee)

Updated

2 years ago
Blocks: 1355895

Comment 4

2 years ago
Pushed by amccreight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3971646449ae
Don't load TelemetryStartup.js in the content process. r=gfritzsche

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3971646449ae
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.