Remove static_casts from PermissionUmaUtil histogram calls.

Remove static_casts from PermissionUmaUtil histogram calls.

Histograms that use enum classes no longer require recorded values to be
static_cast'd to int. This CL removes static_cast<int> calls in
permissions code.

Histogram tests still require explicit cast out of enum classes, so
these are not removed by this CL.

BUG=638076
Review-Url: https://codereview.chromium.org/2791123002
Cr-Commit-Position: refs/heads/master@{#461369}
Committed: https://chromium.googlesource.com/chromium/src/+/669662b980b9fc6e91c8c525e6a0f4e14ab002c5

[dominickn, 2017-04-03 00:17:05]

The CQ bit was checked by dominickn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-03 00:17:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dominickn, 2017-04-03 00:17:32]

dominickn@chromium.org changed reviewers:
+ raymes@chromium.org

[dominickn, 2017-04-03 00:17:33]

PTAL, thanks!

[commit-bot: I haz the power, 2017-04-03 01:03:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-03 01:03:17]

Dry run: This issue passed the CQ dry run.

[raymes, 2017-04-03 03:53:07]

lgtm, thanks for the cleanup :) 

I'm not quite sure why the casts aren't needed in tests - is it just because the
API is a bit different?

[dominickn, 2017-04-03 03:59:21]

The CQ bit was checked by dominickn@chromium.org

[commit-bot: I haz the power, 2017-04-03 03:59:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dominickn, 2017-04-03 04:00:10]

On 2017/04/03 03:53:07, raymes (OOO until April 10) wrote:
> lgtm, thanks for the cleanup :) 
> 
> I'm not quite sure why the casts aren't needed in tests - is it just because
the
> API is a bit different?

Thanks! The CL that allowed enum classes to be used in the histogram calls
didn't also refactor the histogram tests (crrev.com/2776853002).

[commit-bot: I haz the power, 2017-04-03 04:04:52]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491191961113630, "parent_rev":
"021c7e1715272a2fdf05030874e4b76ca3e340bd", "commit_rev":
"669662b980b9fc6e91c8c525e6a0f4e14ab002c5"}

[dominickn, 2017-04-03 04:05:30]

Description was changed from

==========
Remove static_casts from PermissionUmaUtil histogram calls.

Histograms that use enum classes no longer require recorded values to be
static_cast'd to int. This CL removes static_cast<int> calls in
permissions code.

Histogram tests still require explicit cast out of enum classes, so
these are not removed by this CL.

BUG=638076
==========

to

==========
Remove static_casts from PermissionUmaUtil histogram calls.

Histograms that use enum classes no longer require recorded values to be
static_cast'd to int. This CL removes static_cast<int> calls in
permissions code.

Histogram tests still require explicit casts out of enum classes, so
these are not removed by this CL.

BUG=638076
==========

[commit-bot: I haz the power, 2017-04-03 04:06:01]

Description was changed from

==========
Remove static_casts from PermissionUmaUtil histogram calls.

Histograms that use enum classes no longer require recorded values to be
static_cast'd to int. This CL removes static_cast<int> calls in
permissions code.

Histogram tests still require explicit casts out of enum classes, so
these are not removed by this CL.

BUG=638076
==========

to

==========
Remove static_casts from PermissionUmaUtil histogram calls.

Histograms that use enum classes no longer require recorded values to be
static_cast'd to int. This CL removes static_cast<int> calls in
permissions code.

Histogram tests still require explicit cast out of enum classes, so
these are not removed by this CL.

BUG=638076

Review-Url: https://codereview.chromium.org/2791123002
Cr-Commit-Position: refs/heads/master@{#461369}
Committed:
https://chromium.googlesource.com/chromium/src/+/669662b980b9fc6e91c8c525e6a0...
==========

[commit-bot: I haz the power, 2017-04-03 04:06:05]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/669662b980b9fc6e91c8c525e6a0...