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

Side by Side Diff: components/previews/core/previews_experiments.h

Issue 2439203002: Adding a short blacklist period after every previews opt out (Closed)
Patch Set: 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_ 5 #ifndef COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_
6 #define COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_ 6 #define COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_
7 7
8 #include "base/time/time.h" 8 #include "base/time/time.h"
9 9
10 namespace previews { 10 namespace previews {
11 11
12 namespace params { 12 namespace params {
13 13
14 // The maximum number of recent previews navigations the black list looks at to 14 // The maximum number of recent previews navigations the black list looks at to
15 // determine if a host is blacklisted. 15 // determine if a host is blacklisted.
16 size_t MaxStoredHistoryLengthForBlackList(); 16 size_t MaxStoredHistoryLengthForBlackList();
17 17
18 // The maximum number of hosts allowed in the in memory black list. 18 // The maximum number of hosts allowed in the in memory black list.
19 size_t MaxInMemoryHostsInBlackList(); 19 size_t MaxInMemoryHostsInBlackList();
20 20
21 // The number of recent navigations that were opted out of that would trigger 21 // The number of recent navigations that were opted out of that would trigger
22 // the host to be blacklisted. 22 // the host to be blacklisted.
23 int BlackListOptOutThreshold(); 23 int BlackListOptOutThreshold();
24 24
25 // The amount of time a host remains blacklisted due to opt outs. 25 // The amount of time a host remains blacklisted due to opt outs.
26 base::TimeDelta BlackListDuration(); 26 base::TimeDelta BlackListDuration();
27 27
28 // The amount of time after any opt out that no previews should be shown.
29 base::TimeDelta SingleOptOutBlackOutDuration();
tbansal1 2016/10/21 23:15:34 Just call it SingleOptOutDuration?
RyanSturm 2016/10/24 21:18:25 Done.
30
28 } // namespace params 31 } // namespace params
29 32
30 // Returns true if any client-side previews experiment is active. 33 // Returns true if any client-side previews experiment is active.
31 bool IsIncludedInClientSidePreviewsExperimentsFieldTrial(); 34 bool IsIncludedInClientSidePreviewsExperimentsFieldTrial();
32 35
33 // Returns true if the field trial that should enable offline pages for 36 // Returns true if the field trial that should enable offline pages for
34 // prohibitvely slow networks is active. 37 // prohibitvely slow networks is active.
35 bool IsOfflinePreviewsEnabled(); 38 bool IsOfflinePreviewsEnabled();
36 39
37 // Sets the appropriate state for field trial and variations to imitate the 40 // Sets the appropriate state for field trial and variations to imitate the
38 // offline pages field trial. 41 // offline pages field trial.
39 bool EnableOfflinePreviewsForTesting(); 42 bool EnableOfflinePreviewsForTesting();
40 43
41 } // namespace previews 44 } // namespace previews
42 45
43 #endif // COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_ 46 #endif // COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698