[Mac] Tweak crash logging for -sendEvent: crash.

[Mac] Tweak crash logging for -sendEvent: crash.

https://chromiumcodereview.appspot.com/11140040 adds the event's
-description as a breakpad key.  Unfortunately, event type 28, which
is undocumented, is sent by Mountain Lion when you do a control-scroll
zoom.  Work around it manually.

BUG=154483, 156103
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=162397

[Scott Hess - ex-Googler, 2012-10-16 21:56:32]

I had hoped to delete this code, but I haven't found any (relevant) crashes with
the crash key set.

https://codereview.chromium.org/11198008/diff/1/chrome/browser/chrome_browser...
File chrome/browser/chrome_browser_application_mac.mm (right):

https://codereview.chromium.org/11198008/diff/1/chrome/browser/chrome_browser...
chrome/browser/chrome_browser_application_mac.mm:441: base::mac::ScopedCrashKey
key(kSendEventKey, value);
Why not haz magic constant?  Because I couldn't think of a name. 
kUndocumentedCrashingType?  I don't know.

[Robert Sesek, 2012-10-16 21:58:05]

lovez it lgtm

[Scott Hess - ex-Googler, 2012-10-16 22:20:40]

No balk from rsesek, so maybe OWNERS LGTM from thakis?

[Nico, 2012-10-16 22:23:30]

lgtm

https://codereview.chromium.org/11198008/diff/1/chrome/browser/chrome_browser...
File chrome/browser/chrome_browser_application_mac.mm (right):

https://codereview.chromium.org/11198008/diff/1/chrome/browser/chrome_browser...
chrome/browser/chrome_browser_application_mac.mm:441: base::mac::ScopedCrashKey
key(kSendEventKey, value);
On 2012/10/16 21:56:32, shess wrote:
> Why not haz magic constant?  Because I couldn't think of a name. 
> kUndocumentedCrashingType?  I don't know.

kTwentyEight, duh.

[commit-bot: I haz the power, 2012-10-16 22:35:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shess@chromium.org/11198008/1

[commit-bot: I haz the power, 2012-10-17 11:20:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shess@chromium.org/11198008/1

[commit-bot: I haz the power, 2012-10-17 14:17:12]

Change committed as 162397

[Scott Hess - ex-Googler, 2012-10-17 14:55:06]

On 2012/10/16 22:35:38, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
> https://chromium-status.appspot.com/cq/shess%40chromium.org/11198008/1

On 2012/10/17 11:20:10, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
> https://chromium-status.appspot.com/cq/shess%40chromium.org/11198008/1

Well, I guess that missed the canary, then.

[Nico, 2012-10-17 17:29:37]

On 2012/10/17 14:55:06, shess wrote:
> On 2012/10/16 22:35:38, I haz the power (commit-bot) wrote:
> > CQ is trying da patch. Follow status at
> > https://chromium-status.appspot.com/cq/shess%2540chromium.org/11198008/1
> 
> On 2012/10/17 11:20:10, I haz the power (commit-bot) wrote:
> > CQ is trying da patch. Follow status at
> > https://chromium-status.appspot.com/cq/shess%2540chromium.org/11198008/1
> 
> Well, I guess that missed the canary, then.

Urgent crashfix CLs should probably not be landed through the cq, for this
reason.