Closed
Bug 1054493
Opened 10 years ago
Closed 10 years ago
[e10s] Make docshell charset/character set telemetry work
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
e10s | + | --- |
People
(Reporter: adw, Assigned: jimm)
References
Details
Bug 999293 made Firefox's charset menu work in e10s, but it copied docShell.gatherCharsetMenuTelemetry() from browser.js (now in browser.xml) to browser-child.js: https://hg.mozilla.org/mozilla-central/annotate/ba7754e8a224/toolkit/content/browser-child.js#l359
Telemetry is disabled in child processes, so that line ends up doing nothing, and it should be fixed somehow. I don't see a reason why this telemetry has to be done in the docshell. Seems like doing it from the browser chrome JS, where the charset menu code is, would cover the real-word cases where docshells' charsets are changed, although I guess mobile and other apps would need to do the same thing if they wanted the telemetry.
Note that this isn't a regression because before bug 999293 was fixed, the charset menu didn't work in e10s at all.
Updated•10 years ago
|
tracking-e10s:
--- → ?
Assignee | ||
Updated•10 years ago
|
Depends on: 1024201, old-e10s-m2
Assignee | ||
Updated•10 years ago
|
Updated•10 years ago
|
Blocks: old-e10s-m2
No longer depends on: old-e10s-m2
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jmathies
Assignee | ||
Comment 2•10 years ago
|
||
Drew, we have telemetry reporting in the child now (bug 1024021), and I've confirmed that gatherCharsetMenuTelemetry() is getting called in the child in response to an char encoding change in the parent through the text encoding menus.. so I think we can close this out. You mentioned in comment 1 that you had some issues with the way this was put together. Should I morph this into a bug about the location of gatherCharsetMenuTelemetry() or just resolve this wfm?
Either way, I don't think we need to block our milestone 6 on this anymore.
Flags: needinfo?(adw)
Reporter | ||
Comment 3•10 years ago
|
||
Cool, I think my concern in comment 0 is only academic now.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(adw)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•