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

Unified Diff: components/previews/core/previews_experiments.h

Issue 2442013003: Add non-host functionality to the previews blacklist (Closed)
Patch Set: rebase and test Created 4 years, 1 month 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: components/previews/core/previews_experiments.h
diff --git a/components/previews/core/previews_experiments.h b/components/previews/core/previews_experiments.h
index 410e163924b3a59147c9f1eb846d00eb6567b17a..5930a39df6823eb6a68afe8c4715eec797a16fd5 100644
--- a/components/previews/core/previews_experiments.h
+++ b/components/previews/core/previews_experiments.h
@@ -6,31 +6,42 @@
#define COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_
#include "base/time/time.h"
namespace previews {
namespace params {
// The maximum number of recent previews navigations the black list looks at to
// determine if a host is blacklisted.
-size_t MaxStoredHistoryLengthForBlackList();
+size_t MaxStoredHistoryLengthForPerHostBlackList();
+
+// The maximum number of recent previews navigations the black list looks at to
+// determine if all previews navigations are disallowed.
+size_t MaxStoredHistoryLengthForHostIndifferentBlackList();
// The maximum number of hosts allowed in the in memory black list.
size_t MaxInMemoryHostsInBlackList();
+// The number of recent navigations that were opted out of for a given host that
+// would trigger that host to be blacklisted.
+int PerHostBlackListOptOutThreshold();
+
// The number of recent navigations that were opted out of that would trigger
-// the host to be blacklisted.
-int BlackListOptOutThreshold();
+// all previews navigations to be disallowed.
+int HostIndifferentBlackListOptOutThreshold();
// The amount of time a host remains blacklisted due to opt outs.
-base::TimeDelta BlackListDuration();
+base::TimeDelta PerHostBlackListDuration();
+
+// The amount of time all previews navigations are disallowed due to opt outs.
+base::TimeDelta HostIndifferentBlackListPerHostDuration();
// The amount of time after any opt out that no previews should be shown.
base::TimeDelta SingleOptOutDuration();
} // namespace params
enum class PreviewsType {
NONE = 0,
OFFLINE = 1,
LAST = 2,
« no previous file with comments | « components/previews/core/previews_black_list_unittest.cc ('k') | components/previews/core/previews_experiments.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698