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

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

Issue 2442013003: Add non-host functionality to the previews blacklist (Closed)
Patch Set: tbansal comments Created 4 years, 2 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: components/previews/core/previews_experiments.h
diff --git a/components/previews/core/previews_experiments.h b/components/previews/core/previews_experiments.h
index 9c8d7add0574d9a8f0a42d6a9bcdd75d41866221..36e4dd7494dc4f74416ca7ffe8c9d3c03d373505 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 that would trigger
// the host to be blacklisted.
-int BlackListOptOutThreshold();
+int PerHostBlackListOptOutThreshold();
+
+// The number of recent navigations that were opted out of that would trigger
+// the 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();
} // namespace params
// Returns true if any client-side previews experiment is active.
bool IsIncludedInClientSidePreviewsExperimentsFieldTrial();
// Returns true if the field trial that should enable offline pages for
// prohibitvely slow networks is active.
bool IsOfflinePreviewsEnabled();

Powered by Google App Engine
This is Rietveld 408576698