1521801 - mark content process crashes as "content" rather than "tab"

Status: RESOLVED FIXED

(Core :: IPC, defect)

Description

[Nathan Froyd [:froydnj]]

...as I inadvertently changed the behavior in bug 1514044.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: fix process type annotation for content process crashes

This is the minimal solution.  I didn't fix the other inadvertent change in the
telemetry recording because in the regressing bug, haik and bobowen said it
wasn't a problem for that particular bit to change.

Comment 2

[Nathan Froyd [:froydnj]]

It's possible that I really meant to change the enum to use "content" rather than "tab", and that was why we went through the nonsense of asking people if it was OK if telemetry got broken...but given that I screwed things up already, I think the minimal fix is better, especially in soft freeze, and we can go back and make everything "content" later.

Comment 3

[Gabriele Svelto [:gsvelto]]

Comment on attachment 9038260 [details] [diff] [review]
fix process type annotation for content process crashes

Let's go with this quick fix, it's the safest option. This is something I should really have spotted during review :(

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/85b9c2b3965f
fix process type annotation for content process crashes; r=gsvelto

Comment 5

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/85b9c2b3965f