Replace GetAllErrorCodesForUma calls with UMA_HISTOGRAM_SPARSE_SLOWLY.

Replace GetAllErrorCodesForUma calls with UMA_HISTOGRAM_SPARSE_SLOWLY.

Per review comment on https://codereview.chromium.org/1127623005/, sparse
histograms are preferred.

BUG=none

Committed: https://crrev.com/ad6fc44cec8b05ff4ef34f9d38e578b08673646e
Cr-Commit-Position: refs/heads/master@{#330417}

[davidben, 2015-05-15 20:36:08]

davidben@chromium.org changed reviewers:
+ asvitkine@chromium.org

[davidben, 2015-05-15 20:36:08]

As promised.

[Alexei Svitkine (slow), 2015-05-15 20:38:28]

lgtm % comments, thanks for the clean up!

https://codereview.chromium.org/1140273002/diff/1/components/error_page/rende...
File components/error_page/renderer/net_error_helper_core.cc (right):

https://codereview.chromium.org/1140273002/diff/1/components/error_page/rende...
components/error_page/renderer/net_error_helper_core.cc:20: #include
"base/metrics/histogram.h"
Nit: While you're here, can you update this to histogram_macros.h?

https://codereview.chromium.org/1140273002/diff/1/content/browser/download/do...
File content/browser/download/download_resource_handler.cc (right):

https://codereview.chromium.org/1140273002/diff/1/content/browser/download/do...
content/browser/download/download_resource_handler.cc:12: #include
"base/metrics/histogram.h"
Ditto.

https://codereview.chromium.org/1140273002/diff/1/net/dns/host_resolver_impl.cc
File net/dns/host_resolver_impl.cc (right):

https://codereview.chromium.org/1140273002/diff/1/net/dns/host_resolver_impl....
net/dns/host_resolver_impl.cc:26: #include "base/metrics/histogram.h"
Ditto.

[davidben, 2015-05-15 20:41:00]

davidben@chromium.org changed reviewers:
+ mmenke@chromium.org

[davidben, 2015-05-15 20:41:01]

+mmenke for components/error_page/renderer/net_error_helper_core.cc

https://codereview.chromium.org/1140273002/diff/1/components/error_page/rende...
File components/error_page/renderer/net_error_helper_core.cc (right):

https://codereview.chromium.org/1140273002/diff/1/components/error_page/rende...
components/error_page/renderer/net_error_helper_core.cc:20: #include
"base/metrics/histogram.h"
On 2015/05/15 20:38:27, Alexei Svitkine wrote:
> Nit: While you're here, can you update this to histogram_macros.h?

Done.

https://codereview.chromium.org/1140273002/diff/1/content/browser/download/do...
File content/browser/download/download_resource_handler.cc (right):

https://codereview.chromium.org/1140273002/diff/1/content/browser/download/do...
content/browser/download/download_resource_handler.cc:12: #include
"base/metrics/histogram.h"
On 2015/05/15 20:38:27, Alexei Svitkine wrote:
> Ditto.

Done.

https://codereview.chromium.org/1140273002/diff/1/net/dns/host_resolver_impl.cc
File net/dns/host_resolver_impl.cc (right):

https://codereview.chromium.org/1140273002/diff/1/net/dns/host_resolver_impl....
net/dns/host_resolver_impl.cc:26: #include "base/metrics/histogram.h"
On 2015/05/15 20:38:28, Alexei Svitkine wrote:
> Ditto.

Done.

[mmenke, 2015-05-18 14:29:22]

components/error_page/ LGTM.

[davidben, 2015-05-18 17:40:37]

The CQ bit was checked by davidben@chromium.org

[davidben, 2015-05-18 17:40:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/1140273002/#ps20001
(title: "asvitkine comment")

[commit-bot: I haz the power, 2015-05-18 17:41:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1140273002/20001

[commit-bot: I haz the power, 2015-05-18 20:59:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-18 22:43:14]

Patchset 2 (id:??) landed as
https://crrev.com/ad6fc44cec8b05ff4ef34f9d38e578b08673646e
Cr-Commit-Position: refs/heads/master@{#330417}