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

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

Issue 2335023002: Adding a previews IO-thread blacklist (Closed)
Patch Set: tbansal comments Created 4 years, 3 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_black_list_item.h
diff --git a/components/previews/core/previews_black_list_item.h b/components/previews/core/previews_black_list_item.h
new file mode 100644
index 0000000000000000000000000000000000000000..462e61d1c4ade6782ada52f7035baef9dd35d5b2
--- /dev/null
+++ b/components/previews/core/previews_black_list_item.h
@@ -0,0 +1,80 @@
+// 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 COMPONENTS_PREVIEWS_CORE_PREVIEWS_BLACK_LIST_ITEM_H_
+#define COMPONENTS_PREVIEWS_CORE_PREVIEWS_BLACK_LIST_ITEM_H_
+
+#include <stdint.h>
+
+#include <list>
+#include <map>
+#include <memory>
+#include <string>
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "base/optional.h"
+#include "base/time/time.h"
+
+namespace previews {
+
+// Stores the recent black list history for a single host. Stores
+// |stored_history_length| of the most recent previews navigations. To determine
+// previews eligiblity fewer than |opt_out_black_list_threshold| out of the past
+// |stored_history_length| navigations must be opt outs. |black_list_duration|
+// is the amount of time that elapses until the host is no longer on the black
+// list.
+class PreviewsBlackListItem {
+ public:
+ PreviewsBlackListItem(size_t stored_history_length,
+ int opt_out_black_list_threshold,
+ base::TimeDelta black_list_duration);
+
+ ~PreviewsBlackListItem();
+
+ // Adds a new navigation at the specified |entry_time|.
+ void AddPreviewNavigation(bool opt_out, base::Time entry_time);
+
+ // Whether the host name corresponding to |this| should be disallowed from
+ // showing previews.
+ bool IsBlackListed(base::Time now) const;
+
+ base::Optional<base::Time> most_recent_opt_out_time() const {
+ return most_recent_opt_out_time_;
+ }
+
+ private:
+ // A previews navigation to this host is represented by time and whether the
+ // navigation was an opt out.
+ struct OptOutRecord {
+ OptOutRecord(base::Time entry_time, bool opt_out);
+ ~OptOutRecord() {}
+ const base::Time
+ entry_time; // The time that the opt out state was determined.
+ const bool opt_out; // Whether the user opt out of the preview.
+ };
+
+ // The number of entries to store to determine preview eligibility.
+ const size_t max_stored_history_length_;
+ // The number opt outs in recent history that will trigger blacklisting.
+ const int opt_out_black_list_threshold_;
+ // The amount of time to black list a domain after the most recent opt out.
+ const base::TimeDelta max_black_list_duration_;
+
+ // The |max_stored_history_length_| most recent previews navigation. Is
+ // maintained as a list sorted by entry_time (earliest to latest).
+ std::list<OptOutRecord> opt_out_records_;
+
+ // Time of the most recent opt out.
+ base::Optional<base::Time> most_recent_opt_out_time_;
+
+ // The total number of opt outs currently in |opt_out_records_|.
+ int total_opt_out_;
+
+ DISALLOW_COPY_AND_ASSIGN(PreviewsBlackListItem);
+};
+
+} // namespace previews
+
+#endif // COMPONENTS_PREVIEWS_CORE_PREVIEWS_BLACK_LIST_ITEM_H_
« no previous file with comments | « components/previews/core/previews_black_list.cc ('k') | components/previews/core/previews_black_list_item.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698