Closed Bug 696437 Opened 13 years ago Closed 13 years ago

Add Gfx AppNotes for mobile

Categories

(Core :: Graphics, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: BenWa, Assigned: BenWa)

References

Details

Attachments

(1 file)

      No description provided.
Testing:
Run with '--es env0 MOZ_CRASHREPORTER=1'
adb shell run-as org.mozilla.fennec_benoitgirard kill -6 <PID>
(6 is SIGABRT)
Attached patch patchSplinter Review
I wonder if we should also print the GL extensions?
Attachment #571002 - Flags: review?(bjacob)
Sample report:
http://crash-stats.mozilla.com/report/index/bp-c9df5898-8cad-4431-ae15-a153f2111101
Assignee: nobody → bgirard
Status: NEW → ASSIGNED
Comment on attachment 571002 [details] [diff] [review]
patch

Review of attachment 571002 [details] [diff] [review]:
-----------------------------------------------------------------

r=me provided that you remove those debug printfs.

::: widget/src/android/GfxInfo.cpp
@@ +222,2 @@
>  {
> +  printf_stderr("Crash notes start\n");

remove those printfs, they look like a leftover from debugging.
Attachment #571002 - Flags: review?(bjacob) → review+
https://hg.mozilla.org/mozilla-central/rev/98892c213475
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: