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

Unified Diff: chrome/browser/permissions/permission_decision_auto_blocker.cc

Issue 2250893002: Permission Action Reporting: Add num_prior_* fields. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@par-new-fields-proto
Patch Set: Also update browsertest Created 4 years, 4 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/permissions/permission_decision_auto_blocker.cc
diff --git a/chrome/browser/permissions/permission_decision_auto_blocker.cc b/chrome/browser/permissions/permission_decision_auto_blocker.cc
index 12e7170e22ef149eafa9af2888c58d500bd6c523..2d94b7e65c0a195d00833c175dbca20f084ad9b4 100644
--- a/chrome/browser/permissions/permission_decision_auto_blocker.cc
+++ b/chrome/browser/permissions/permission_decision_auto_blocker.cc
@@ -119,7 +119,17 @@ bool PermissionDecisionAutoBlocker::ShouldChangeDismissalToBlock(
return current_dismissal_count >= prompt_dismissals_before_block_;
}
-int PermissionDecisionAutoBlocker::GetActionCountForTest(
+int PermissionDecisionAutoBlocker::GetDismissCount(const GURL& url,
+ content::PermissionType permission) {
+ return GetActionCount(url, permission, kPromptDismissCountKey);
+}
+
+int PermissionDecisionAutoBlocker::GetIgnoreCount(const GURL& url,
+ content::PermissionType permission) {
+ return GetActionCount(url, permission, kPromptIgnoreCountKey);
+}
+
+int PermissionDecisionAutoBlocker::GetActionCount(
const GURL& url,
content::PermissionType permission,
const char* key) {

Powered by Google App Engine
This is Rietveld 408576698