Closed
Bug 1345894
Opened 7 years ago
Closed 7 years ago
Extend plugin notification/infobar telemetry through Flash CTP rollout
Categories
(Core Graveyard :: Plug-ins, enhancement)
Core Graveyard
Plug-ins
Tracking
(firefox55 fixed)
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: benjamin, Assigned: benjamin)
References
Details
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
Details |
The plugin notification telemetry is currently marked expires: never, but we don't actually need it forever now that we have a Flash CTP rollout plan. Also it's *not* opt-out, but we really want that for release telemetry. So we're going to mark expiration past the schedule for Flash CTP rollout, and make it opt-out. This was originally added in bug 902075.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
Comment on attachment 8845459 [details] Bug 1345894 - Set expiry for plugin notification/infobar telemetry and make it opt-out for monitoring Flash CTP rollout, data-r=rweiss For some reason reviewboard won't let me set r?rweiss@mozilla.com in the UI, so I'm setting it here.
Attachment #8845459 -
Flags: review?(rweiss)
Comment 3•7 years ago
|
||
Just to clarify, this will be opt-out in Release but only for a finite period of time? If so, that should clear me for review.
Assignee | ||
Comment 4•7 years ago
|
||
Yes: "releaseChannelCollection": "opt-out", "expires_in_version": "60",
Comment 5•7 years ago
|
||
Comment on attachment 8845459 [details] Bug 1345894 - Set expiry for plugin notification/infobar telemetry and make it opt-out for monitoring Flash CTP rollout, data-r=rweiss SGTM.
Attachment #8845459 -
Flags: review?(rweiss) → review+
Assignee | ||
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8845459 [details] Bug 1345894 - Set expiry for plugin notification/infobar telemetry and make it opt-out for monitoring Flash CTP rollout, data-r=rweiss https://reviewboard.mozilla.org/r/118620/#review124634 rweiss marked r+ in bugzilla
Attachment #8845459 -
Flags: review+
Pushed by bsmedberg@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/ec18ee811b9a Set expiry for plugin notification/infobar telemetry and make it opt-out for monitoring Flash CTP rollout, data-r=rweiss
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/mozilla-inbound/rev/6ad5c0f245e2 Set expiry for plugin notification/infobar telemetry and make it opt-out for monitoring Flash CTP rollout: Follow-up: Remove histograms from whitelist of email-less histograms. r=bustage-fix on a CLOSED TREE
Builds were still busted after the followup, backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/6d48bea7160d
https://treeherder.mozilla.org/logviewer.html#?job_id=85446394&repo=mozilla-inbound
Flags: needinfo?(benjamin)
Updated•7 years ago
|
Assignee: nobody → benjamin
Comment hidden (mozreview-request) |
Comment 12•7 years ago
|
||
Pushed by bsmedberg@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/24d718f4a6c2 Set expiry for plugin notification/infobar telemetry and make it opt-out for monitoring Flash CTP rollout, data-r=rweiss
Comment 13•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/24d718f4a6c2
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(benjamin)
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•