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

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

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
« no previous file with comments | « components/previews/core/previews_experiments.h ('k') | components/previews/core/previews_opt_out_store.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/previews/core/previews_experiments.cc
diff --git a/components/previews/core/previews_experiments.cc b/components/previews/core/previews_experiments.cc
index 9e55f510a50f46d0734d0bcfd2870cf30d37c0cb..97ab3e9a0783b0317bd5adf3c015f5acdd9c0685 100644
--- a/components/previews/core/previews_experiments.cc
+++ b/components/previews/core/previews_experiments.cc
@@ -20,105 +20,142 @@ namespace {
// The group of client-side previews experiments.
const char kClientSidePreviewsFieldTrial[] = "ClientSidePreviews";
const char kEnabled[] = "Enabled";
// Allow offline pages to show for prohibitively slow networks.
const char kOfflinePagesSlowNetwork[] = "show_offline_pages";
// The maximum number of recent previews navigations the black list looks at to
// determine if a host is blacklisted.
-const char kMaxStoredHistoryLength[] = "stored_history_length";
+const char kMaxStoredHistoryLengthPerHost[] =
+ "per_host_max_stored_history_length";
+
+// The maximum number of recent previews navigations the black list looks at to
+// determine if all previews navigations should be disallowed.
+const char kMaxStoredHistoryLengthHostIndifferent[] =
+ "host_indifferent_max_stored_history_length";
// The maximum number of hosts allowed in the in memory black list.
const char kMaxHostsInBlackList[] = "max_hosts_in_blacklist";
// The number of recent navigations that were opted out of that would trigger
// the host to be blacklisted.
-const char kOptOutThreshold[] = "opt_out_threshold";
+const char kPerHostOptOutThreshold[] = "per_host_opt_out_threshold";
+
+// The number of recent navigations that were opted out of that would trigger
+// all previews navigations to be disallowed.
+const char kHostIndifferentOptOutThreshold[] =
+ "host_indifferent_opt_out_threshold";
// The amount of time a host remains blacklisted due to opt outs.
-const char kBlackListDurationInDays[] = "black_list_duration_in_days";
+const char kPerHostBlackListDurationInDays[] =
+ "per_host_black_list_duration_in_days";
+
+// The amount of time a host remains blacklisted due to opt outs.
+const char kHostIndifferentBlackListDurationInDays[] =
+ "host_indifferent_black_list_duration_in_days";
// The amount of time after any opt out that no previews should be shown.
const char kSingleOptOutDurationInSeconds[] =
"single_opt_out_duration_in_seconds";
// The string that corresponds to enabled for the variation param experiments.
const char kExperimentEnabled[] = "true";
-// In seconds. Hosts are blacklisted for 30 days.
-const int kDefaultBlackListDurationInDays = 30;
-
-// In seconds. Previews are not shown for 5 minutes after an opt out.
-constexpr int kDefaultSingleOptOutDurationInSeconds = 60 * 5;
-
// Returns the parameter value of |param| as a string. If there is no value for
// |param|, returns an empty string.
std::string ParamValue(const std::string& param) {
if (!IsIncludedInClientSidePreviewsExperimentsFieldTrial())
return std::string();
std::map<std::string, std::string> experiment_params;
if (!variations::GetVariationParams(kClientSidePreviewsFieldTrial,
&experiment_params)) {
return std::string();
}
std::map<std::string, std::string>::const_iterator it =
experiment_params.find(param);
return it == experiment_params.end() ? std::string() : it->second;
}
} // namespace
namespace params {
-size_t MaxStoredHistoryLengthForBlackList() {
- std::string param_value = ParamValue(kMaxStoredHistoryLength);
+size_t MaxStoredHistoryLengthForPerHostBlackList() {
+ std::string param_value = ParamValue(kMaxStoredHistoryLengthPerHost);
size_t history_length;
if (!base::StringToSizeT(param_value, &history_length)) {
return 4;
}
return history_length;
}
+size_t MaxStoredHistoryLengthForHostIndifferentBlackList() {
+ std::string param_value = ParamValue(kMaxStoredHistoryLengthHostIndifferent);
+ size_t history_length;
+ if (!base::StringToSizeT(param_value, &history_length)) {
+ return 10;
+ }
+ return history_length;
+}
+
size_t MaxInMemoryHostsInBlackList() {
std::string param_value = ParamValue(kMaxHostsInBlackList);
size_t max_hosts;
if (!base::StringToSizeT(param_value, &max_hosts)) {
return 100;
}
return max_hosts;
}
-int BlackListOptOutThreshold() {
- std::string param_value = ParamValue(kOptOutThreshold);
+int PerHostBlackListOptOutThreshold() {
+ std::string param_value = ParamValue(kPerHostOptOutThreshold);
int opt_out_threshold;
if (!base::StringToInt(param_value, &opt_out_threshold)) {
return 2;
}
return opt_out_threshold;
}
-base::TimeDelta BlackListDuration() {
- std::string param_value = ParamValue(kBlackListDurationInDays);
+int HostIndifferentBlackListOptOutThreshold() {
+ std::string param_value = ParamValue(kHostIndifferentOptOutThreshold);
+ int opt_out_threshold;
+ if (!base::StringToInt(param_value, &opt_out_threshold)) {
+ return 4;
+ }
+ return opt_out_threshold;
+}
+
+base::TimeDelta PerHostBlackListDuration() {
+ std::string param_value = ParamValue(kPerHostBlackListDurationInDays);
+ int duration;
+ if (!base::StringToInt(param_value, &duration)) {
+ return base::TimeDelta::FromDays(30);
+ }
+ return base::TimeDelta::FromDays(duration);
+}
+
+base::TimeDelta HostIndifferentBlackListPerHostDuration() {
+ std::string param_value = ParamValue(kHostIndifferentBlackListDurationInDays);
int duration;
if (!base::StringToInt(param_value, &duration)) {
- return base::TimeDelta::FromDays(kDefaultBlackListDurationInDays);
+ return base::TimeDelta::FromDays(365 * 100);
}
return base::TimeDelta::FromDays(duration);
}
base::TimeDelta SingleOptOutDuration() {
std::string param_value = ParamValue(kSingleOptOutDurationInSeconds);
int duration;
if (!base::StringToInt(param_value, &duration)) {
- return base::TimeDelta::FromSeconds(kDefaultSingleOptOutDurationInSeconds);
+ return base::TimeDelta::FromSeconds(60 * 5);
}
return base::TimeDelta::FromSeconds(duration);
}
} // namespace params
bool IsIncludedInClientSidePreviewsExperimentsFieldTrial() {
// By convention, an experiment in the client-side previews study enables use
// of at least one client-side previews optimization if its name begins with
// "Enabled."
« no previous file with comments | « components/previews/core/previews_experiments.h ('k') | components/previews/core/previews_opt_out_store.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698