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

Unified Diff: components/previews/core/previews_opt_out_store.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_opt_out_store.h
diff --git a/components/previews/core/previews_opt_out_store.h b/components/previews/core/previews_opt_out_store.h
new file mode 100644
index 0000000000000000000000000000000000000000..c1d1bc80fd2232353b3611b791a27ef855a0e1e2
--- /dev/null
+++ b/components/previews/core/previews_opt_out_store.h
@@ -0,0 +1,59 @@
+// 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_OPT_OUT_STORE_H_
+#define COMPONENTS_PREVIEWS_CORE_PREVIEWS_OPT_OUT_STORE_H_
+
+#include <stdint.h>
+
+#include <memory>
+#include <string>
+#include <unordered_map>
+#include <vector>
+
+#include "base/callback.h"
+#include "base/time/time.h"
+#include "components/previews/core/previews_black_list_item.h"
+
+class GURL;
+
+namespace previews {
+
+enum class PreviewsType {
+ NONE = 0,
+ OFFLINE = 1,
+ LAST = 2,
+};
+
+typedef std::unordered_map<std::string, std::unique_ptr<PreviewsBlackListItem>>
+ BlackListItemMap;
+
+typedef base::Callback<void(std::unique_ptr<BlackListItemMap>)>
+ LoadBlackListCallback;
+
+// PreviewsOptOutStore keeps opt out information for the previews.
+// Ability to create multiple instances of the store as well as behavior of
+// asynchronous operations when the object is being destroyed, before such
+// operation finishes will depend on implementation. It is possible to issue
+// multiple asynchronous operations in parallel and maintain ordering.
+class PreviewsOptOutStore {
+ public:
+ PreviewsOptOutStore();
+ virtual ~PreviewsOptOutStore();
tbansal1 2016/09/20 17:48:52 You can remove the constructor. Inline the destruc
RyanSturm 2016/09/20 18:38:17 Done.
+
+ // Adds a new navigation to the store. |opt_out| is whether the uesr opted out
+ // of the preview or navigated away from the page by another way.
+ virtual void AddPreviewNavigation(bool opt_out,
+ const std::string& host_name,
+ PreviewsType type,
+ base::Time now) = 0;
+
+ // Asynchronously loads a map of host names to PreviewsBlackListItem for that
+ // host from the store.
+ virtual void LoadBlackList(LoadBlackListCallback callback) = 0;
+};
+
+} // namespace previews
+
+#endif // COMPONENTS_PREVIEWS_CORE_PREVIEWS_OPT_OUT_STORE_H_

Powered by Google App Engine
This is Rietveld 408576698