Closed
Bug 606539
Opened 14 years ago
Closed 14 years ago
Report Android specific info in crash reports
Categories
(Toolkit :: Crash Reporting, enhancement)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fennec | 2.0b2+ | --- |
People
(Reporter: mwu, Assigned: blassey)
Details
Attachments
(1 file, 2 obsolete files)
2.43 KB,
patch
|
blassey
:
review+
ted
:
review+
blassey
:
approval2.0+
|
Details | Diff | Splinter Review |
It would be nice to get Android specific info in a crash report or even just know at a glance that a crash report is for android.
Comment 1•14 years ago
|
||
The current OS field is Linux, then? Do you just want to change that to Android instead? I don't think grouping Android and Linux-desktop reports makes a lot of sense in general. What other "Android specific info" are you talking about? Is it something that you want to be able to query on crash-stats? Reporting metadata is easy, but using it is hard.
Comment 2•14 years ago
|
||
Right now the OS field comes from the minidump. Since we're using the Linux dumper code, it reports itself as Linux. I guess we could change that and add a separate "Android" value and make it use that instead. I think that'd require changes in all of: http://code.google.com/p/google-breakpad/source/browse/trunk/src/google_breakpad/common/minidump_format.h#599 http://code.google.com/p/google-breakpad/source/browse/trunk/src/processor/minidump.cc#1747 http://code.google.com/p/google-breakpad/source/browse/trunk/src/processor/minidump.cc#3081 http://code.google.com/p/google-breakpad/source/browse/trunk/src/processor/minidump_processor.cc#388 http://code.google.com/p/google-breakpad/source/browse/trunk/src/processor/minidump_processor.cc#828 http://code.google.com/p/google-breakpad/source/browse/trunk/src/client/linux/minidump_writer/minidump_writer.cc#1169
Assignee | ||
Comment 3•14 years ago
|
||
we could also post-process that in the client, which would probably be easier
Comment 4•14 years ago
|
||
How? That data is part of the minidump, not the text file.
Assignee | ||
Comment 5•14 years ago
|
||
oh sorry, I misread ted's comment
Reporter | ||
Comment 6•14 years ago
|
||
We would also like info on the device name and build type (thumb2 or nothumb)
Comment 7•14 years ago
|
||
That we might have a harder time getting into the minidump, although we could probably report CPU features, since there's space in the CPU field where we stick CPUID data for x86.
Assignee | ||
Comment 8•14 years ago
|
||
this sends a fair bit of extra info along as extra keys, Tt also appends the device manufacturer, device model and build figure print to the notes field since that's more readable in the crash reports. If the min cpu is less than 7 it also appends "nonthumb2 build". Here is an example crash report with this change http://crash-stats.mozilla.com/report/index/bp-36e1a5c0-5001-49da-a701-78d4c2101027
Assignee: nobody → blassey.bugs
Attachment #486383 -
Flags: review?(ted.mielczarek)
Attachment #486383 -
Flags: review?(mwu)
Reporter | ||
Comment 9•14 years ago
|
||
Comment on attachment 486383 [details] [diff] [review] patch All the fields in Build are static, so you don't need to instantiate a Build object. Eliminate the extra empty line after sendPart(os, boundary, kNotesKey, notes); Add a space after notes in the nonthumb2 build part. Lets use nothumb instead of nonthumb2 to keep naming consistency with ftp.
Attachment #486383 -
Flags: review?(mwu)
Assignee | ||
Comment 10•14 years ago
|
||
Attachment #486383 -
Attachment is obsolete: true
Attachment #486459 -
Flags: review?(mwu)
Attachment #486383 -
Flags: review?(ted.mielczarek)
Reporter | ||
Comment 11•14 years ago
|
||
Comment on attachment 486459 [details] [diff] [review] patch r=me with the last comment in the last review fixed. That is, there should be a space in notes+= and s/nonthumb2/nothumb/.
Attachment #486459 -
Flags: review?(mwu) → review+
Assignee | ||
Comment 12•14 years ago
|
||
Comment on attachment 486459 [details] [diff] [review] patch got approval for fennec b2 from Stuart over irc
Attachment #486459 -
Flags: approval2.0+
Assignee | ||
Comment 13•14 years ago
|
||
Attachment #486459 -
Attachment is obsolete: true
Attachment #486480 -
Flags: review+
Attachment #486480 -
Flags: approval2.0+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 14•14 years ago
|
||
Comment on attachment 486480 [details] [diff] [review] patch for checkin Use carriage returns to separate the bits you're putting in the notes field. Also, can you use a more descriptive name for "Min CPU"? (also you used a space in that name where you used underscores for the rest, should probably be consistent.) Please file a followup bug to get Socorro to display these additional fields, and put a comment above where you set the Notes field referencing it, so that we can remove that code once Socorro can display the other fields.
Attachment #486480 -
Flags: review+
Assignee | ||
Updated•14 years ago
|
tracking-fennec: --- → 2.0b2+
Assignee | ||
Comment 15•14 years ago
|
||
pushed http://hg.mozilla.org/mozilla-central/rev/3369bc46ff9c and filed bug 607942
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•