Closed
Bug 1442763
Opened 6 years ago
Closed 6 years ago
Set correct flag for "off by default in release"' and only send UT telemetry when general telemetry is sent
Categories
(Firefox :: New Tab Page, enhancement, P1)
Firefox
New Tab Page
Tracking
()
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: emtwo, Assigned: emtwo)
References
Details
Attachments
(1 file)
This is a follow up to Bug 1440378 and has 2 parts. 1) We should be using AppConstants.EARLY_BETA_OR_EARLIER instead of this: https://github.com/mozilla/activity-stream/blob/master/system-addon/lib/ActivityStream.jsm#L121 2) sendUTEvent() should check for "eventEnabled" in addition to "eventTelemetryEnabled"
Assignee | ||
Comment 1•6 years ago
|
||
2) Refers to this: https://github.com/mozilla/activity-stream/blob/master/system-addon/lib/TelemetryFeed.jsm#L363
Comment 2•6 years ago
|
||
Comment 3•6 years ago
|
||
Commit pushed to master at https://github.com/mozilla/activity-stream https://github.com/mozilla/activity-stream/commit/d4c5fa7eb6f54b29f08069b313ffa0a8556eca7a Bug 1442763: Set correct flag for 'off by default in release' and only send UT telemetry when general telemetry is sent. (#4017)
Updated•6 years ago
|
Severity: normal → enhancement
Iteration: --- → 60.4 - Mar 12
status-firefox60:
--- → affected
Priority: -- → P1
Updated•6 years ago
|
Comment 4•6 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9e902a91c375
Target Milestone: --- → Firefox 60
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•