Adds PreviewsType version mechanism for clearing blacklist entries.

components/previews/core/previews_experiments.cc

 // 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.
 
 #include "components/previews/core/previews_experiments.h"
 
 #include <map>
 #include <string>
 
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "components/variations/variations_associated_data.h"
 
 namespace previews {
 
 namespace {
 
 // The group of client-side previews experiments.
 const char kClientSidePreviewsFieldTrial[] = "ClientSidePreviews";
 
 const char kEnabled[] = "Enabled";
 
 // Allow offline pages to show for prohibitively slow networks.
 const char kOfflinePagesSlowNetwork[] = "show_offline_pages";
 
+// Version of treatment for showing offline pages, if enabled.
+const char kOfflinePagesSlowNetworkVersion[] = "show_offline_pages.version";
+
 // The maximum number of recent previews navigations the black list looks at to
 // determine if a host is blacklisted.
 const char kMaxStoredHistoryLengthPerHost[] =
     "per_host_max_stored_history_length";
 
 // The maximum number of recent previews navigations the black list looks at to
 // determine if all previews navigations should be disallowed.
 const char kMaxStoredHistoryLengthHostIndifferent[] =
     "host_indifferent_max_stored_history_length";
 
@@ skipping 134 matching lines @@
 bool IsPreviewsTypeEnabled(PreviewsType type) {
   switch (type) {
     case PreviewsType::OFFLINE:
       return ParamValue(kOfflinePagesSlowNetwork) == kExperimentEnabled;
     default:
       NOTREACHED();
       return false;
   }
 }
 
+bool GetPreviewsTypeVersion(PreviewsType type) {
+  DCHECK(IsPreviewsTypeEnabled(type));
+  int version = 0;  // default
+  switch (type) {
+    case PreviewsType::OFFLINE:
+      base::StringToInt(ParamValue(kOfflinePagesSlowNetworkVersion), &version);
+      return version;

[tbansal1, 2017/01/26 00:59:14]

I think the switch-case can be rewritten to get compile-time checks:

switch (type) {
 case PreviewsType::OFFLINE:
   return blah;
 case PreviewsType::NONE:
 case PreviewsType::OFFLINE:
        NOTREACHED();
       return 0;
}

See
https://groups.google.com/a/chromium.org/forum/#!searchin/chromium-dev/switch...

[dougarnett, 2017/02/07 21:15:02]

Done.

+    default:
+      NOTREACHED();
+  }
+  return version;
+}
+
+std::unique_ptr<std::vector<std::pair<PreviewsType, int>>>
+GetEnabledPreviews() {
+  std::unique_ptr<std::vector<std::pair<PreviewsType, int>>> enabled_previews(
+      new std::vector<std::pair<PreviewsType, int>>());
+
+  if (IsPreviewsTypeEnabled(PreviewsType::OFFLINE)) {
+    enabled_previews->push_back(
+        {PreviewsType::OFFLINE, GetPreviewsTypeVersion(PreviewsType::OFFLINE)});
+  }
+  return enabled_previews;
+}
+
 bool EnableOfflinePreviewsForTesting() {
   std::map<std::string, std::string> params;
   params[kOfflinePagesSlowNetwork] = kExperimentEnabled;
   return variations::AssociateVariationParams(kClientSidePreviewsFieldTrial,
                                               kEnabled, params) &&
          base::FieldTrialList::CreateFieldTrial(kClientSidePreviewsFieldTrial,
                                                 kEnabled);
 }
 
 }  // namespace previews