Closed Bug 839456 Opened 12 years ago Closed 12 years ago

do some in-process crash annotation with plugin versions on Android

Categories

(Core Graveyard :: Plug-ins, defect, P2)

Product:
Core Graveyard
Component:
Plug-ins
Platform:
ARM
Android
Type:
defect

Tracking

(firefox20 fixed, firefox21 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla21
Tracking Flags:
Tracking Status
firefox20 --- fixed
firefox21 --- fixed

People

(Reporter: blassey, Assigned: snorp)

References

Details

Attachments

(1 file, 3 obsolete files)

Annotate crash reports with Flash version on Android
(deleted), patch
snorp
: review+
lsblakk
: approval-mozilla-aurora+
lsblakk
: approval-mozilla-beta-
Details | Diff | Splinter Review
This will help us figure out if crashes we're seeing are from antiquated versions of flash and blocking them might be a useful solution.
If we can't get the version from the plugin library, we can get it from PackageInfo: http://developer.android.com/reference/android/content/pm/PackageInfo.html#versionCode
As I recently discussed with bsmedberg, we apparently have added something like this for plugin processes, but unfortunately we don't run plugins on Android out-of-process and therefore don't get that. Benjamin, can we get something in there despite that?
Of course, just annotate the Flash version when you first load it using nsICrashReporter.annotateCrashReport("FlashVersion", versionstring).
Attachment #711975 - Flags: review?(benjamin)
Attachment #711975 - Attachment is obsolete: true
Attachment #711975 - Flags: review?(benjamin)
Attachment #711977 - Flags: review?(benjamin)
Can't build with crash reporter for Fennec on Mac, so untested. Try run here: https://tbpl.mozilla.org/?tree=Try&rev=7cc5fe856f26
(In reply to Benjamin Smedberg [:bsmedberg] from comment #3) > Of course, just annotate the Flash version when you first load it using > nsICrashReporter.annotateCrashReport("FlashVersion", versionstring). Is it ok to do this more than once? The patch above does it on every instance.
(In reply to James Willcox (:snorp) (jwillcox@mozilla.com) from comment #7) > (In reply to Benjamin Smedberg [:bsmedberg] from comment #3) > > Of course, just annotate the Flash version when you first load it using > > nsICrashReporter.annotateCrashReport("FlashVersion", versionstring). > > Is it ok to do this more than once? The patch above does it on every > instance. It looks like a hashtable, so guess it's fine.
Comment on attachment 711977 [details] [diff] [review] Annotate crash reports with Flash version on Android Assuming you've verified that pluginTag->mVersion contains useful data, sure.
Attachment #711977 - Flags: review?(benjamin) → review+
Attachment #711977 - Attachment is obsolete: true
Comment on attachment 712482 [details] [diff] [review] Annotate crash reports with Flash version on Android Wrapped "FlashVersion" in NS_LITERAL_CSTRING so it should build/work now. Carrying forward r+ from prior patch. Try run here: https://tbpl.mozilla.org/?tree=Try&rev=ba45f4126a18
Attachment #712482 - Flags: review+
Priority: -- → P2
Attachment #712482 - Attachment is obsolete: true
Comment on attachment 713056 [details] [diff] [review] Annotate crash reports with Flash version on Android Third time is a charm. Try run here: https://tbpl.mozilla.org/?tree=Try&rev=9074ad2fe504
Attachment #713056 - Flags: review+
Depends on: 840634
Comment on attachment 713056 [details] [diff] [review] Annotate crash reports with Flash version on Android [Approval Request Comment] I want to make sure this is working on m-c first, but we should merge to Aurora and Beta pronto.
Attachment #713056 - Flags: approval-mozilla-beta?
Attachment #713056 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/796564dd007b
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment on attachment 713056 [details] [diff] [review] Annotate crash reports with Flash version on Android As much as we'd like this on Beta we're now past the point of taking any non-critical landings on that branch. Approving for Aurora only.
Attachment #713056 - Flags: approval-mozilla-beta?
Attachment #713056 - Flags: approval-mozilla-beta-
Attachment #713056 - Flags: approval-mozilla-aurora?
Attachment #713056 - Flags: approval-mozilla-aurora+
Blocks: 794265
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: