predictors: Move the configuration of speculative prefetch to field trials.

chrome/browser/predictors/resource_prefetch_common.cc

 // Copyright 2014 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 "chrome/browser/predictors/resource_prefetch_common.h"
 
 #include <string>
 #include <tuple>
 
 #include "base/command_line.h"
+#include "base/metrics/field_trial.h"
 #include "chrome/browser/net/prediction_options.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sessions/session_tab_helper.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/prefs/pref_service.h"
+#include "components/variations/variations_associated_data.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
 
 namespace predictors {
 
+const char kSpeculativeResourcePrefetchingFeatureName[] =
+    "SpeculativeResourcePrefetching";
+const char kModeParamName[] = "mode";
+const char kLearningMode[] = "learning";
+const char kExternalPrefetchingMode[] = "external-prefetching";
+const char kPrefetchingMode[] = "prefetching";
+
 namespace {
 
+const base::Feature kSpeculativeResourcePrefetchingFeature{
+    kSpeculativeResourcePrefetchingFeatureName,
+    base::FEATURE_DISABLED_BY_DEFAULT};
+
 bool IsPrefetchingEnabledInternal(Profile* profile, int mode, int mask) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   if ((mode & mask) == 0)
     return false;
 
   if (!profile || !profile->GetPrefs() ||
       chrome_browser_net::CanPrefetchAndPrerenderUI(profile->GetPrefs()) !=
           chrome_browser_net::NetworkPredictionStatus::ENABLED) {
     return false;
   }
 
   return true;
 }
 
 }  // namespace
 
 bool IsSpeculativeResourcePrefetchingEnabled(
     Profile* profile,
     ResourcePrefetchPredictorConfig* config) {
   DCHECK(config);
 
-  // Off the record - disabled.
+  // Disabled for of-the-record. Policy choice, not a technical limitation.
   if (!profile || profile->IsOffTheRecord())
     return false;
 
-  // Enabled by command line switch. The config has the default params already
-  // set. The command line with just enable them with the default params.
-  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kSpeculativeResourcePrefetching)) {
-    const std::string value =
-        base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
-            switches::kSpeculativeResourcePrefetching);
+  if (!base::FeatureList::IsEnabled(kSpeculativeResourcePrefetchingFeature))
+    return false;
 
-    if (value == switches::kSpeculativeResourcePrefetchingDisabled) {
-      return false;
-    } else if (value == switches::kSpeculativeResourcePrefetchingLearning) {
-      config->mode |= ResourcePrefetchPredictorConfig::LEARNING;
-      return true;
-    } else if (value ==
-               switches::kSpeculativeResourcePrefetchingEnabledExternal) {
-      config->mode |= ResourcePrefetchPredictorConfig::LEARNING |
-                      ResourcePrefetchPredictorConfig::PREFETCHING_FOR_EXTERNAL;
-      return true;
-    } else if (value == switches::kSpeculativeResourcePrefetchingEnabled) {
-      config->mode |=
-          ResourcePrefetchPredictorConfig::LEARNING |
-          ResourcePrefetchPredictorConfig::PREFETCHING_FOR_NAVIGATION |
-          ResourcePrefetchPredictorConfig::PREFETCHING_FOR_EXTERNAL;
-      return true;
-    }
+  std::string mode_value = variations::GetVariationParamValueByFeature(
+      kSpeculativeResourcePrefetchingFeature, kModeParamName);
+  if (mode_value == kLearningMode) {
+    config->mode |= ResourcePrefetchPredictorConfig::LEARNING;
+  } else if (mode_value == kExternalPrefetchingMode) {
+    config->mode |= ResourcePrefetchPredictorConfig::LEARNING |
+                    ResourcePrefetchPredictorConfig::PREFETCHING_FOR_EXTERNAL;
+  } else if (mode_value == kPrefetchingMode) {
+    config->mode |= ResourcePrefetchPredictorConfig::LEARNING |
+                    ResourcePrefetchPredictorConfig::PREFETCHING_FOR_EXTERNAL |
+                    ResourcePrefetchPredictorConfig::PREFETCHING_FOR_NAVIGATION;
+  } else {
+    return false;

[droger, 2017/01/25 17:27:11]

Nit:
For readability it may be better to have a return statement in all the branches
or none.

[Benoit L, 2017/01/26 09:58:15]

Done.

   }
 
-  return false;
+  return true;
 }
 
 NavigationID::NavigationID() : tab_id(-1) {}
 
 NavigationID::NavigationID(const NavigationID& other)
     : tab_id(other.tab_id),
       main_frame_url(other.main_frame_url),
       creation_time(other.creation_time) {}
 
 NavigationID::NavigationID(content::WebContents* web_contents)
@@ skipping 82 matching lines @@
 
 bool ResourcePrefetchPredictorConfig::IsMoreResourcesEnabledForTest() const {
   return max_resources_per_entry == 100;
 }
 
 bool ResourcePrefetchPredictorConfig::IsSmallDBEnabledForTest() const {
   return max_urls_to_track == 200 && max_hosts_to_track == 100;
 }
 
 }  // namespace predictors