Adding a short blacklist period after every previews opt out

components/previews/core/previews_black_list.h

 // 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_H_
 #define COMPONENTS_PREVIEWS_CORE_PREVIEWS_BLACK_LIST_H_
 
 #include <stdint.h>
 
 #include <memory>
 #include <queue>
 #include <string>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "base/optional.h"
 #include "base/threading/thread_checker.h"
 #include "components/previews/core/previews_opt_out_store.h"
 
 class GURL;
 
 namespace base {
 class Clock;

[tbansal1, 2016/10/25 00:47:41]

#include "base/time/time.h"
and remove this.

(otherwise, you will need to forward declare base::Time).

[RyanSturm, 2016/10/25 17:15:38]

I definitely need base/time/time.h, but I also need base/time/clock or the
forward declare.

 }
 
 namespace previews {
 class PreviewsBlackListItem;
 
 // Manages the state of black listed domains for the previews experiment. Loads
 // the stored black list from |opt_out_store| and manages an in memory black
 // list on the IO thread. Updates to the black list are stored in memory and
 // pushed to the store. Asynchronous modifications are stored in a queue and
 // executed in order. Reading from the black list is always synchronous, and if
@@ skipping 61 matching lines @@
   // Enqueues a task to run when when loading black list information has
   // completed. Maintains the order that tasks were called in.
   void QueuePendingTask(base::Closure callback);
 
   // Map maintaining the in-memory black list.
   std::unique_ptr<BlackListItemMap> black_list_item_map_;
 
   // Whether the black list is done being loaded from the backing store.
   bool loaded_;
 
+  // The time of the last opt out for this session.
+  base::Optional<base::Time> last_opt_out_time_;
+
   // The backing store of the black list information.
   std::unique_ptr<PreviewsOptOutStore> opt_out_store_;
 
   // Callbacks to be run after loading information from the backing store has
   // completed.
   std::queue<base::Closure> pending_callbacks_;
 
   std::unique_ptr<base::Clock> clock_;
 
   base::ThreadChecker thread_checker_;
 
   base::WeakPtrFactory<PreviewsBlackList> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PreviewsBlackList);
 };
 
 }  // namespace previews
 
 #endif  // COMPONENTS_PREVIEWS_CORE_PREVIEWS_BLACK_LIST_H_