Revert "Adding UMA to track previews opt outs and blacklist eligibility"

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>
@@ skipping 12 matching lines @@
 
 class GURL;
 
 namespace base {
 class Clock;
 }
 
 namespace previews {
 class PreviewsBlackListItem;
 
-enum class PreviewsEligibilityReason {
-  // The preview navigation was allowed.
-  ALLOWED = 0,
-  // The black list was not initialized.
-  BLACKLIST_UNAVAILABLE = 1,
-  // The black list has not loaded from disk yet.
-  BLACKLIST_DATA_NOT_LOADED = 2,
-  // The user has opted out of a preview recently.
-  USER_RECENTLY_OPTED_OUT = 3,
-  // The user has opted out of previews often, and is no longer shown previews
-  // on any host.
-  USER_BLACKLISTED = 4,
-  // The user has opted out of previews on a specific host often, and was not
-  // not shown a previews on that host.
-  HOST_BLACKLISTED = 5,
-  // The network quality estimate is not available.
-  NETWORK_QUALITY_UNAVAILABLE = 6,
-  // The network was fast enough to not warrant previews.
-  NETWORK_NOT_SLOW = 7,
-  LAST = 8,
-};
-
 // 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
 // the black list is not currently loaded (e.g., at startup, after clearing
 // browsing history), domains are reported as black listed. The list stores no
 // more than previews::params::MaxInMemoryHostsInBlackList hosts in-memory,
 // which defaults to 100.
 class PreviewsBlackList {
  public:
   // |opt_out_store| is the backing store to retrieve and store black list
   // information, and can be null. When |opt_out_store| is null, the in-memory
   // map will be immediately loaded to empty. If |opt_out_store| is non-null,
   // it will be used to load the in-memory map asynchronously.
   PreviewsBlackList(std::unique_ptr<PreviewsOptOutStore> opt_out_store,
                     std::unique_ptr<base::Clock> clock);
   ~PreviewsBlackList();
 
   // Asynchronously adds a new navigation to to the in-memory black list and
   // backing store. |opt_out| is whether the user opted out of the preview or
   // navigated away from the page without opting out. |type| is only passed to
   // the backing store. If the in memory map has reached the max number of hosts
   // allowed, and |url| is a new host, a host will be evicted based on recency
   // of the hosts most recent opt out.
   void AddPreviewNavigation(const GURL& url, bool opt_out, PreviewsType type);
 
   // Synchronously determines if |host_name| should be allowed to show previews.
-  // Returns the reason the blacklist disallowed the preview, or
-  // PreviewsEligibilityReason::ALLOWED if the preview is allowed.
-  PreviewsEligibilityReason IsLoadedAndAllowed(const GURL& url,
-                                               PreviewsType type) const;
+  // If the black list has loaded yet, this will always return false. |type| is
+  // not used to make this decision.
+  bool IsLoadedAndAllowed(const GURL& url, PreviewsType type) const;
 
   // Asynchronously deletes all entries in the in-memory black list. Informs
   // the backing store to delete entries between |begin_time| and |end_time|,
   // and reloads entries into memory from the backing store. If the embedder
   // passed in a null store, resets all history in the in-memory black list.
   void ClearBlackList(base::Time begin_time, base::Time end_time);
 
   // Returns a new PreviewsBlackListItem representing |host_name|. Adds the new
   // item to |black_list_item_map|.
   static PreviewsBlackListItem* GetOrCreateBlackListItemForMap(
@@ skipping 51 matching lines @@
   base::ThreadChecker thread_checker_;
 
   base::WeakPtrFactory<PreviewsBlackList> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PreviewsBlackList);
 };
 
 }  // namespace previews
 
 #endif  // COMPONENTS_PREVIEWS_CORE_PREVIEWS_BLACK_LIST_H_