Adding number of users to crash record

Adding number of users to ChromeOS crash record

BUG=330217
TEST=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245408

[Mr4D (OOO till 08-26), 2014-01-16 15:21:00]

Please have a look!

[Robert Sesek, 2014-01-16 15:34:07]

Can you reupload? Rietveld complains of chunk mismatch.

[Mr4D (OOO till 08-26), 2014-01-16 16:34:17]

Very odd. I didn't get any error message when uploading the patch, so I didn't
check. Done. Please have another look!

[Dmitry Polukhin, 2014-01-16 17:17:15]

lgtm

[Robert Sesek, 2014-01-16 17:26:00]

On 2014/01/16 16:34:17, Mr4D wrote:
> Very odd. I didn't get any error message when uploading the patch, so I didn't
> check. Done. Please have another look!

The mysteries of Rietveld…

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
File chrome/common/crash_keys.cc (right):

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
chrome/common/crash_keys.cc:67: #if defined(OS_CHROMEOS)
Same.

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
chrome/common/crash_keys.cc:123: #if defined(OS_CHROMEOS)
Same.

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_keys.h
File chrome/common/crash_keys.h (right):

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
chrome/common/crash_keys.h:84: #if defined(OS_CHROMEOS)
Can you move this to be before or the Mac section on line 112, so that
platform-specific ones are grouped near each other?

[Mr4D (OOO till 08-26), 2014-01-16 17:51:02]

Addressed. Please have another look!

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
File chrome/common/crash_keys.cc (right):

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
chrome/common/crash_keys.cc:67: #if defined(OS_CHROMEOS)
On 2014/01/16 17:26:00, rsesek wrote:
> Same.

Done.

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
chrome/common/crash_keys.cc:123: #if defined(OS_CHROMEOS)
On 2014/01/16 17:26:00, rsesek wrote:
> Same.

Done.

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_keys.h
File chrome/common/crash_keys.h (right):

https://codereview.chromium.org/137753007/diff/270001/chrome/common/crash_key...
chrome/common/crash_keys.h:84: #if defined(OS_CHROMEOS)
On 2014/01/16 17:26:00, rsesek wrote:
> Can you move this to be before or the Mac section on line 112, so that
> platform-specific ones are grouped near each other?

Done.

[Robert Sesek, 2014-01-16 17:52:42]

lgtm

[commit-bot: I haz the power, 2014-01-16 17:55:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skuhne@chromium.org/137753007/380001

[commit-bot: I haz the power, 2014-01-16 23:01:04]

Retried try job too often on win_rel for step(s) base_unittests, browser_tests,
interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2014-01-16 23:06:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skuhne@chromium.org/137753007/380001

[commit-bot: I haz the power, 2014-01-17 01:34:46]

Change committed as 245408