Chromium Code Reviews| Index: components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc |
| diff --git a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc |
| index 593095a8d802e33a09accbee6e41b5c222206dd3..0f40f1586b863ccff2e45f206babe60ed864862d 100644 |
| --- a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc |
| +++ b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc |
| @@ -354,6 +354,7 @@ int ContentSubresourceFilterDriverFactory::CalculateHitPatternForActivationList( |
| const int kInitialURLHitMask = 0x4; |
| const int kRedirectURLHitMask = 0x2; |
| const int kFinalURLHitMask = 0x1; |
| + |
| if (navigation_chain_.size() > 1) { |
| if (DidURLMatchActivationList(navigation_chain_.back(), activation_list)) |
| hits_pattern |= kFinalURLHitMask; |
| @@ -381,6 +382,7 @@ void ContentSubresourceFilterDriverFactory::RecordRedirectChainMatchPattern() |
| RecordRedirectChainMatchPatternForList( |
| ActivationList::SOCIAL_ENG_ADS_INTERSTITIAL); |
| RecordRedirectChainMatchPatternForList(ActivationList::PHISHING_INTERSTITIAL); |
| + RecordRedirectChainMatchPatternForList(ActivationList::SUBRESOURCE_FILTER); |
| } |
| void ContentSubresourceFilterDriverFactory:: |
| @@ -399,6 +401,10 @@ void ContentSubresourceFilterDriverFactory:: |
| REPORT_REDIRECT_PATTERN_FOR_SUFFIX("PhishingInterstital", hits_pattern, |
| chain_size); |
| break; |
| + case ActivationList::SUBRESOURCE_FILTER: |
| + REPORT_REDIRECT_PATTERN_FOR_SUFFIX("SubresourceFilterOnly", hits_pattern, |
|
vakh (use Gerrit instead)
2017/03/16 20:51:18
s/Only// ?
melandory
2017/03/24 15:49:27
I like "Only" in suffix, without it full histogram
|
| + chain_size); |
| + break; |
| default: |
| NOTREACHED(); |
| break; |