1054493 - [e10s] Make docshell charset/character set telemetry work

Status: RESOLVED WORKSFORME

(Firefox :: General, defect)

Description

[Drew Willcoxon :adw]

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.

Comment 1

[Chris Peterson [:cpeterson]]

Move old M2's low-priority bugs to M6 milestone.

Comment 2

[Jim Mathies [:jimm]]

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.

Comment 3

[Drew Willcoxon :adw]

Cool, I think my concern in comment 0 is only academic now.