Added check for invalid histogram data, added additional diagnostics to find the source of troubles…

Added check for invalid histogram data, added additional diagnostics to find the source of troubles on ChromeOS.

BUG=140301
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=149797

[hshi1, 2012-08-03 02:24:23]

LGTM with nit (indentation), thanks!

https://chromiumcodereview.appspot.com/10825169/diff/1/chrome/browser/chromeo...
File chrome/browser/chromeos/external_metrics.cc (right):

https://chromiumcodereview.appspot.com/10825169/diff/1/chrome/browser/chromeo...
chrome/browser/chromeos/external_metrics.cc:42: size_t bucket_count) {
nit: please vertically left-align the arguments

[zel, 2012-08-03 02:29:45]

https://chromiumcodereview.appspot.com/10825169/diff/1/chrome/browser/chromeo...
File chrome/browser/chromeos/external_metrics.cc (right):

https://chromiumcodereview.appspot.com/10825169/diff/1/chrome/browser/chromeo...
chrome/browser/chromeos/external_metrics.cc:42: size_t bucket_count) {
On 2012/08/03 02:24:24, hshi1 wrote:
> nit: please vertically left-align the arguments

Done.

[commit-bot: I haz the power, 2012-08-03 02:30:15]

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

[commit-bot: I haz the power, 2012-08-03 03:14:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zelidrag@chromium.org/10825169/3

[commit-bot: I haz the power, 2012-08-03 07:18:46]

Change committed as 149797

[jar (doing other things), 2012-08-24 00:17:26]

lgtm