Closed
Bug 843586
Opened 12 years ago
Closed 12 years ago
Add telemetry flag for "character encoding menu used"
Categories
(Core :: DOM: Navigation, defect)
Core
DOM: Navigation
Tracking
()
RESOLVED
DUPLICATE
of bug 840476
People
(Reporter: hsivonen, Assigned: hsivonen)
References
Details
Attachments
(1 file)
|
2.08 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
To gather data about whether/when we can retire the character encoding menu, we should add a telemetry flag for measuring if it has been used in the browser session.
|
Assignee | |
Comment 1•12 years ago
|
||
|
|
Assignee | |
Updated•12 years ago
|
Flags: needinfo?(sstamm)
Keywords: privacy-review-needed
|
||
Comment 2•12 years ago
|
||
If it's just a "yes/no" about whether or not it's been activated, this is pretty clearly measuring feature usage and is in scope. Please flag curtisk for future privacy reviews.
Curtis: do you need to do anything else here?
Flags: needinfo?(sstamm) → needinfo?(curtisk)
Keywords: privacy-review-needed
other than adding this to the chart for telemetry (https://wiki.mozilla.org/Privacy/Reviews/Telemetry/Measurements) no I don't think so.
Flags: needinfo?(curtisk)
|
||
Comment 4•12 years ago
|
||
Comment on attachment 716532 [details] [diff] [review]
Add the flag
Ok, this is just a flag.
Attachment #716532 -
Flags: review?(bugs) → review+
|
|
||
Comment 5•12 years ago
|
||
As discussed on IRC, the probe doesn't exactly handle only menu UI, since addons may and do
use docshell.charset.
So if better probe is needed, it should go to browser UI code.
|
|
Assignee | |
Comment 6•12 years ago
|
||
Folded this into the new patch for bug 840476 which addresses the add-on issue.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•