Add a feature which, when enabled, blocks permissions after X prompt dismissals.

chrome/browser/permissions/permission_prompt_decision_log.h

Index: chrome/browser/permissions/permission_prompt_decision_log.h
diff --git a/chrome/browser/permissions/permission_prompt_decision_log.h b/chrome/browser/permissions/permission_prompt_decision_log.h
new file mode 100644
index 0000000000000000000000000000000000000000..480804adee29d0c6873bfc3c6ab3f9fa1563d2ec
--- /dev/null
+++ b/chrome/browser/permissions/permission_prompt_decision_log.h
@@ -0,0 +1,49 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_PROMPT_DECISION_LOG_H_
+#define CHROME_BROWSER_PERMISSIONS_PERMISSION_PROMPT_DECISION_LOG_H_
+
+#include "base/callback_forward.h"
+#include "base/macros.h"
+#include "content/public/browser/permission_type.h"
+#include "url/gurl.h"
+
+class GURL;
+class Profile;
+
+class PermissionPromptDecisionLog {

[raymes, 2016/08/05 03:27:48]

Sorry this is nitpicky but I think this name will confuse people (it sounds too
general). How about we call it PermissionDecisionAutoBlocker or something to do
with the fact that it will be blocking?

[dominickn, 2016/08/05 04:29:30]

Done.

+ public:
+  static const char kPromptDismissCountKey[];

[raymes, 2016/08/05 03:27:48]

nit: this can be private and the test is a friend anyway.

[dominickn, 2016/08/05 04:29:30]

Done.

+
+  static void RemoveCountsByUrl(Profile* profile,
+                                base::Callback<bool(const GURL& url)> filter);

[raymes, 2016/08/05 03:27:48]

Since this object is a singleton, I'd suggest that all the functions in it could
just be member functions without harm

[dominickn, 2016/08/05 04:29:30]

RemoveCountsByUrl is called from the BrowsingDataRemover, which doesn't have any
way to get to this object. I think that method needs to be static. Otherwise
updated.

Also, this required some changes to the unittest to get right.

+
+  PermissionPromptDecisionLog();
+  bool ShouldChangeDismissalToBlock(Profile* profile,
+                                    const GURL& url,
+                                    content::PermissionType permission);
+
+ private:
+  friend class PermissionContextBaseTests;
+  friend class PermissionPromptDecisionLogUnitTest;
+
+  // Returns the current number of dismissals for |permission| type at |url|.
+  // Exposed for testing.
+  static int GetDismissalCount(Profile* profile,
+                               const GURL& url,
+                               content::PermissionType permission);
+
+  // Records that the user dismissed a permission prompt for type |permission|
+  // on |url| to a website setting. Returns the updated number of dismissals.
+  static int RecordDismissalCount(Profile* profile,
+                                  const GURL& url,
+                                  content::PermissionType permission);
+
+  void UpdateFromVariations();
+
+  int prompt_dismissals_before_block_;
+};
+
+#endif  // CHROME_BROWSER_PERMISSIONS_PERMISSION_PROMPT_DECISION_LOG_H_