Adds PreviewsType version mechanism for clearing blacklist entries.

components/previews/core/previews_experiments.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_EXPERIMENTS_H_
 #define COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_
 
+#include <memory>
+#include <utility>
+#include <vector>
+
 #include "base/time/time.h"
 #include "net/nqe/effective_connection_type.h"
 
 namespace previews {
 
 namespace params {
 
 // The maximum number of recent previews navigations the black list looks at to
 // determine if a host is blacklisted.
 size_t MaxStoredHistoryLengthForPerHostBlackList();
@@ skipping 38 matching lines @@
   LAST = 2,
 };
 
 // Returns true if any client-side previews experiment is active.
 bool IsIncludedInClientSidePreviewsExperimentsFieldTrial();
 
 // Returns true if the field trial that should enable previews for |type| for
 // prohibitvely slow networks is active.
 bool IsPreviewsTypeEnabled(PreviewsType type);
 
+// Returns the version of preview treatment |type| to use if it is enabled.
+// Should only be called if |IsPreviewsTypeEnabled(type)| returns true.

[bengr, 2017/02/09 17:33:47]

Why? Isn't a version independent of whether it is enabled?

[dougarnett, 2017/02/13 17:56:17]

Done.

+int GetPreviewsTypeVersion(PreviewsType type);
+
+// Returns the enabled PreviewsTypes with their version.
+std::unique_ptr<std::vector<std::pair<PreviewsType, int>>> GetEnabledPreviews();

[bengr, 2017/02/09 17:33:47]

Might be more readable if you define a PreviewsTypeList

[dougarnett, 2017/02/13 17:56:17]

Done.

+
 // Sets the appropriate state for field trial and variations to imitate the
 // offline pages field trial.
 bool EnableOfflinePreviewsForTesting();
 
 }  // namespace previews
 
 #endif  // COMPONENTS_PREVIEWS_CORE_PREVIEWS_EXPERIMENTS_H_