Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5119)

Unified Diff: chrome/browser/subresource_filter/subresource_filter_browsertest.cc

Issue 2703093002: Add metrics for tracking subresource filter activation suppression. (Closed)
Patch Set: address comments Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/subresource_filter/subresource_filter_browsertest.cc
diff --git a/chrome/browser/subresource_filter/subresource_filter_browsertest.cc b/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
index 9724eae20670359081a1762dfa4dd5a2dcbc01d0..06b4a14d953b28128d3af890fe94b011f0e22f96 100644
--- a/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
+++ b/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
@@ -30,6 +30,7 @@
#include "components/safe_browsing_db/test_database_manager.h"
#include "components/safe_browsing_db/util.h"
#include "components/security_interstitials/content/unsafe_resource.h"
+#include "components/subresource_filter/content/browser/content_subresource_filter_driver_factory.h"
#include "components/subresource_filter/core/browser/subresource_filter_features.h"
#include "components/subresource_filter/core/browser/subresource_filter_features_test_support.h"
#include "components/subresource_filter/core/common/activation_level.h"
@@ -767,6 +768,8 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterBrowserTest,
ConfigureAsPhishingURL(url);
ASSERT_NO_FATAL_FAILURE(
SetRulesetToDisallowURLsWithPathSuffix("included_script.js"));
+
+ base::HistogramTester tester;
ui_test_utils::NavigateToURL(browser(), url);
EXPECT_FALSE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
@@ -776,6 +779,18 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterBrowserTest,
chrome::Reload(browser(), WindowOpenDisposition::CURRENT_TAB);
observer.Wait();
EXPECT_FALSE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecision, 2);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecision,
+ ContentSubresourceFilterDriverFactory::ACTIVATED, 2);
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload, 1);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload,
+ ContentSubresourceFilterDriverFactory::ACTIVATED, 1);
}
IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
@@ -784,6 +799,8 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
ConfigureAsPhishingURL(url);
ASSERT_NO_FATAL_FAILURE(
SetRulesetToDisallowURLsWithPathSuffix("included_script.js"));
+
+ base::HistogramTester tester;
ui_test_utils::NavigateToURL(browser(), url);
EXPECT_FALSE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
@@ -793,6 +810,21 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
chrome::Reload(browser(), WindowOpenDisposition::CURRENT_TAB);
observer.Wait();
EXPECT_TRUE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecision, 2);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecision,
+ ContentSubresourceFilterDriverFactory::ACTIVATED, 1);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecision,
+ ContentSubresourceFilterDriverFactory::URL_WHITELISTED, 1);
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload, 1);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload,
+ ContentSubresourceFilterDriverFactory::URL_WHITELISTED, 1);
}
IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
@@ -801,6 +833,8 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
ConfigureAsPhishingURL(url);
ASSERT_NO_FATAL_FAILURE(
SetRulesetToDisallowURLsWithPathSuffix("included_script.js"));
+
+ base::HistogramTester tester;
ui_test_utils::NavigateToURL(browser(), url);
EXPECT_FALSE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
@@ -812,6 +846,21 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
"location.reload();"));
observer.Wait();
EXPECT_TRUE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecision, 2);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecision,
+ ContentSubresourceFilterDriverFactory::ACTIVATED, 1);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecision,
+ ContentSubresourceFilterDriverFactory::URL_WHITELISTED, 1);
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload, 1);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload,
+ ContentSubresourceFilterDriverFactory::URL_WHITELISTED, 1);
}
IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
@@ -820,6 +869,8 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
ConfigureAsPhishingURL(url);
ASSERT_NO_FATAL_FAILURE(
SetRulesetToDisallowURLsWithPathSuffix("included_script.js"));
+
+ base::HistogramTester tester;
ui_test_utils::NavigateToURL(browser(), url);
EXPECT_FALSE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
@@ -831,6 +882,21 @@ IN_PROC_BROWSER_TEST_F(SubresourceFilterWhitelistSiteOnReloadBrowserTest,
browser()->tab_strip_model()->GetActiveWebContents(), nav_frame_script));
observer.Wait();
EXPECT_TRUE(WasParsedScriptElementLoaded(web_contents()->GetMainFrame()));
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecision, 2);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecision,
+ ContentSubresourceFilterDriverFactory::ACTIVATED, 1);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecision,
+ ContentSubresourceFilterDriverFactory::URL_WHITELISTED, 1);
+
+ tester.ExpectTotalCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload, 1);
+ tester.ExpectBucketCount(
+ internal::kHistogramSubresourceFilterActivationDecisionReload,
+ ContentSubresourceFilterDriverFactory::URL_WHITELISTED, 1);
}
} // namespace subresource_filter

Powered by Google App Engine
This is Rietveld 408576698