Part 1: Log the `cause` exception that we report to Socorro

Part 2: Still try to report exception if the exception reporter throws an exception.

Comment on attachment 736983 [details] [diff] [review] part-1-log-original-exception.patch Review of attachment 736983 [details] [diff] [review]: ----------------------------------------------------------------- LGTM

Comment on attachment 736984 [details] [diff] [review] part-2-ignore-exception-reporter-exceptions.patch Review of attachment 736984 [details] [diff] [review]: ----------------------------------------------------------------- What case does this catch that wasn't caught before? If an exception is thrown anywhere stackTrace will end up null and so nothing will be reported, so I don't see how this helps.

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #4) > What case does this catch that wasn't caught before? If an exception is > thrown anywhere stackTrace will end up null and so nothing will be reported, > so I don't see how this helps. oops! <:) An earlier version of this patch only had the try {} around the SharedPreferences code because I wondered whether writing SharedPreferences after we just received an OutOfMemoryError could fail with another OutOfMemoryError. It was a purely hypothetical concern.

Comment on attachment 737030 [details] [diff] [review] part-2-ignore-exception-reporter-exceptions-v2.patch r=me with the compile error fixed.

Landed patch 2 with compile error fixed <:) https://hg.mozilla.org/integration/mozilla-inbound/rev/f91e963cad33