Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2777)

Unified Diff: chrome/browser/prerender/prerender_field_trial.cc

Issue 2754003006: Prerender: omnibox and instant field trial parameters. (Closed)
Patch Set: comments Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/prerender/prerender_field_trial.h ('k') | chrome/browser/prerender/prerender_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_field_trial.cc
diff --git a/chrome/browser/prerender/prerender_field_trial.cc b/chrome/browser/prerender/prerender_field_trial.cc
index 8857b8c8d611b44b7fbae4bb9bf627a4e3f39a0a..56835cf42395725b0cc36bfe1dd22f18d48b54cd 100644
--- a/chrome/browser/prerender/prerender_field_trial.cc
+++ b/chrome/browser/prerender/prerender_field_trial.cc
@@ -8,54 +8,83 @@
#include "base/logging.h"
#include "base/metrics/field_trial.h"
+#include "base/metrics/field_trial_params.h"
#include "chrome/browser/prerender/prerender_manager.h"
#include "chrome/common/chrome_switches.h"
-#include "components/variations/variations_associated_data.h"
namespace prerender {
-// NoStatePrefetch feature modes, to control the PrerenderManager mode using
-// the base::Feature API and field trials.
-const char kNoStatePrefetchFeatureModeParameterName[] = "mode";
-const char kNoStatePrefetchFeatureModeParameterPrefetch[] = "no_state_prefetch";
-const char kNoStatePrefetchFeatureModeParameterPrerender[] = "prerender";
-const char kNoStatePrefetchFeatureModeParameterSimpleLoad[] = "simple_load";
-
-const base::Feature kNoStatePrefetchFeature{"NoStatePrefetch",
- base::FEATURE_ENABLED_BY_DEFAULT};
+namespace {
-void ConfigurePrerender() {
- PrerenderManager::PrerenderManagerMode mode =
- PrerenderManager::PRERENDER_MODE_ENABLED;
+PrerenderManager::PrerenderManagerMode ParsePrerenderMode(
+ const char* parameter_name,
+ PrerenderManager::PrerenderManagerMode default_mode) {
+ PrerenderManager::PrerenderManagerMode mode = default_mode;
if (!base::FeatureList::IsEnabled(kNoStatePrefetchFeature)) {
mode = PrerenderManager::PRERENDER_MODE_DISABLED;
} else {
- std::string mode_value = variations::GetVariationParamValueByFeature(
- kNoStatePrefetchFeature, kNoStatePrefetchFeatureModeParameterName);
- if (mode_value == kNoStatePrefetchFeatureModeParameterPrefetch) {
+ std::string mode_value = base::GetFieldTrialParamValueByFeature(
+ kNoStatePrefetchFeature, parameter_name);
+ if (mode_value.empty()) {
+ mode = default_mode;
+ } else if (mode_value == kNoStatePrefetchFeatureModeParameterPrefetch) {
mode = PrerenderManager::PRERENDER_MODE_NOSTATE_PREFETCH;
- } else if (mode_value.empty() ||
- mode_value == kNoStatePrefetchFeatureModeParameterPrerender) {
- // The empty string means the option was provided with no value, and that
- // means enable.
+ } else if (mode_value == kNoStatePrefetchFeatureModeParameterPrerender) {
mode = PrerenderManager::PRERENDER_MODE_ENABLED;
} else if (mode_value == kNoStatePrefetchFeatureModeParameterSimpleLoad) {
mode = PrerenderManager::PRERENDER_MODE_SIMPLE_LOAD_EXPERIMENT;
- } else {
- LOG(ERROR) << "Invalid prerender mode: " << mode_value;
- LOG(ERROR) << "Disabling prerendering!";
+ } else if (mode_value == kNoStatePrefetchFeatureModeParameterDisabled) {
mode = PrerenderManager::PRERENDER_MODE_DISABLED;
+ } else {
+ LOG(ERROR) << "Invalid prerender mode: " << mode_value << " for "
+ << parameter_name;
+ LOG(ERROR) << "Using default mode " << default_mode << " for "
+ << parameter_name << "!";
+ mode = default_mode;
}
}
+ return mode;
+}
+
+} // namespace
- PrerenderManager::SetMode(mode);
+// NoStatePrefetch feature parameters, to control the PrerenderManager mode
+// using the base::Feature API and field trials.
+
+// The general prerender mode for most origins.
+const char kNoStatePrefetchFeatureModeParameterName[] = "mode";
+
+// The origins can have prerendering overridden differently than the rest of the
+// experiment.
+const char kNoStatePrefetchFeatureInstantModeParameterName[] = "instant_mode";
+const char kNoStatePrefetchFeatureOmniboxModeParameterName[] = "omnibox_mode";
+
+// Mode values.
+const char kNoStatePrefetchFeatureModeParameterPrefetch[] = "no_state_prefetch";
+const char kNoStatePrefetchFeatureModeParameterPrerender[] = "prerender";
+const char kNoStatePrefetchFeatureModeParameterSimpleLoad[] = "simple_load";
+const char kNoStatePrefetchFeatureModeParameterDisabled[] = "disabled";
+
+const base::Feature kNoStatePrefetchFeature{"NoStatePrefetch",
+ base::FEATURE_ENABLED_BY_DEFAULT};
+
+void ConfigurePrerender() {
+ PrerenderManager::PrerenderManagerMode overall_mode =
+ ParsePrerenderMode(kNoStatePrefetchFeatureModeParameterName,
+ PrerenderManager::PRERENDER_MODE_ENABLED);
+
+ PrerenderManager::SetMode(overall_mode);
+ PrerenderManager::SetInstantMode(ParsePrerenderMode(
+ kNoStatePrefetchFeatureInstantModeParameterName, overall_mode));
+ PrerenderManager::SetOmniboxMode(ParsePrerenderMode(
+ kNoStatePrefetchFeatureOmniboxModeParameterName, overall_mode));
}
bool IsOmniboxEnabled(Profile* profile) {
if (!profile)
return false;
- if (!PrerenderManager::IsPrerenderingPossible())
+ if (!PrerenderManager::IsAnyPrerenderingPossible())
return false;
// Override any field trial groups if the user has set a command line flag.
« no previous file with comments | « chrome/browser/prerender/prerender_field_trial.h ('k') | chrome/browser/prerender/prerender_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698