Created attachment 8700902 [details] [diff] [review] more-save-telemetry v0.1 It occurred to me that we do not use specific "save.1" probes when saving a page to PDF, saving an image, saving a media file or printing a page. These all seem like "save" operations. They don't have an "unsave.1" counterpart though. This patch adds "save.1" probes and adds comments for why "share.1" probes are not needed in the JS code.
Attachment #8700902 - Flags: review?(margaret.leibovic)
Attachment #8700902 - Flags: review?(margaret.leibovic) → review+
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Comment on attachment 8700902 [details] [diff] [review] more-save-telemetry v0.1 Approval Request Comment [Feature/regressing bug #]: We get a more complete picture of how people save content [User impact if declined]: Less telemetry [Describe test coverage new/current, TreeHerder]: Working on Nightly [Risks and why]: Low [String/UUID change made/needed]: None
Comment on attachment 8700902 [details] [diff] [review] more-save-telemetry v0.1 Been on Nightly for a week, seems safe to uplift to Beta44, Aurora45.
status-firefox44: --- → affected
status-firefox45: --- → affected
status-firefox45: affected → fixed
status-firefox44: affected → fixed
status-b2g-v2.5: --- → fixed
You need to log in before you can comment on or make changes to this bug.