Update FilteringResult UMA to include whitelist information.

Update FilteringResult UMA to include whitelist information.

Add new histogram value for hen a navigation is allowed due to it being
part of a whitelist.

BUG=595640
Committed: https://crrev.com/6d78ae46074968cc7ab3a44b45fdc93941d0f15e
Cr-Commit-Position: refs/heads/master@{#381723}

[atanasova, 2016-03-17 10:48:34]

atanasova@chromium.org changed reviewers:
+ treib@chromium.org

[Marc Treib, 2016-03-17 10:53:14]

https://codereview.chromium.org/1810923002/diff/1/chrome/browser/supervised_u...
File chrome/browser/supervised_user/supervised_user_resource_throttle.cc
(right):

https://codereview.chromium.org/1810923002/diff/1/chrome/browser/supervised_u...
chrome/browser/supervised_user/supervised_user_resource_throttle.cc:57: return
FILTERING_BEHAVIOR_ALLOW_WHITELIST;
This should go under the case for WARN - those are treated the same. WARN isn't
actually used, but still.
(Don't we have a presubmit check or something that complains about the
fallthrough?!)

[atanasova, 2016-03-17 11:13:21]

https://codereview.chromium.org/1810923002/diff/1/chrome/browser/supervised_u...
File chrome/browser/supervised_user/supervised_user_resource_throttle.cc
(right):

https://codereview.chromium.org/1810923002/diff/1/chrome/browser/supervised_u...
chrome/browser/supervised_user/supervised_user_resource_throttle.cc:57: return
FILTERING_BEHAVIOR_ALLOW_WHITELIST;
On 2016/03/17 10:53:14, Marc Treib wrote:
> This should go under the case for WARN - those are treated the same. WARN
isn't
> actually used, but still.
> (Don't we have a presubmit check or something that complains about the
> fallthrough?!)

Done.

[Marc Treib, 2016-03-17 11:14:49]

LGTM, thanks!

[atanasova, 2016-03-17 11:17:24]

atanasova@chromium.org changed reviewers:
+ rkaplow@chromium.org

[atanasova, 2016-03-17 11:17:24]

Adding rkaplow@ for histogram changes approval.

[rkaplow, 2016-03-17 14:42:33]

lgtm

[atanasova, 2016-03-17 14:42:57]

The CQ bit was checked by atanasova@chromium.org

[commit-bot: I haz the power, 2016-03-17 14:43:13]

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

[commit-bot: I haz the power, 2016-03-17 16:07:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-17 16:08:47]

Description was changed from

==========
Update FilteringResult UMA to include whitelist information.

Add new histogram value for hen a navigation is allowed due to it being
part of a whitelist.

BUG=595640
==========

to

==========
Update FilteringResult UMA to include whitelist information.

Add new histogram value for hen a navigation is allowed due to it being
part of a whitelist.

BUG=595640

Committed: https://crrev.com/6d78ae46074968cc7ab3a44b45fdc93941d0f15e
Cr-Commit-Position: refs/heads/master@{#381723}
==========

[commit-bot: I haz the power, 2016-03-17 16:08:48]

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