Launch HQP & HUP score changes.

components/omnibox/browser/history_url_provider.cc

Index: components/omnibox/browser/history_url_provider.cc
diff --git a/components/omnibox/browser/history_url_provider.cc b/components/omnibox/browser/history_url_provider.cc
index 3ad59b5e569fab2814e5f9b735423048612e3536..e136b67e29d094a2d98efeede33ec7683a736872 100644
--- a/components/omnibox/browser/history_url_provider.cc
+++ b/components/omnibox/browser/history_url_provider.cc
@@ -114,23 +114,32 @@ double CalculateRelevanceUsingScoreBuckets(
     const HUPScoringParams::ScoreBuckets& score_buckets,
     const base::TimeDelta& time_since_last_visit,
     int undecayed_relevance,
-    int count) {
+    int undecayed_count) {
   // Back off if above relevance cap.
   if ((score_buckets.relevance_cap() != -1) &&
       (undecayed_relevance >= score_buckets.relevance_cap()))
     return undecayed_relevance;
 
   // Time based decay using half-life time.
-  double decayed_count = count;
+  double decayed_count = undecayed_count;
+  double decay_factor = score_buckets.HalfLifeTimeDecay(time_since_last_visit);
   if (decayed_count > 0)
-    decayed_count *= score_buckets.HalfLifeTimeDecay(time_since_last_visit);
+    decayed_count *= decay_factor;
 
-  // Find a threshold where decayed_count >= bucket.
+  // Find a threshold where decayed_count >= bucket. If buckets are based on
+  // decay factor, then use it correspondingly.
   const HUPScoringParams::ScoreBuckets::CountMaxRelevance* score_bucket = NULL;
   for (size_t i = 0; i < score_buckets.buckets().size(); ++i) {
     score_bucket = &score_buckets.buckets()[i];
-    if (decayed_count >= score_bucket->first)
-      break;  // Buckets are in descending order, so we can ignore the rest.
+    if (score_buckets.use_decay_factor()) {

[Bart N., 2015/08/26 23:01:25]

This is invariant, so I suggest taking it out of the loop.

Something like:
const double factor = (score_buckets.use_decay_factor() ? decay_factor :
decay_count);

and then use 'factor' inside the loop.

[Ashok vardhan, 2015/08/27 00:30:46]

Done.

+      if (decay_factor >= score_bucket->first) {
+        break;
+      }
+    } else {
+      if (decayed_count >= score_bucket->first) {
+        break;
+      }
+    }
   }
 
   return (score_bucket && (undecayed_relevance > score_bucket->second)) ?
@@ -146,8 +155,10 @@ int CalculateRelevanceScoreUsingScoringParams(
     const history::HistoryMatch& match,
     int old_relevance,
     const HUPScoringParams& scoring_params) {
-  if (!scoring_params.experimental_scoring_enabled)
+  if (!scoring_params.experimental_scoring_enabled &&
+      !scoring_params.default_scoring_enabled) {

[Bart N., 2015/08/26 23:01:25]

I don't understand why we need default_scoring_enabled.

Can you simply make default_scoring params initialized before applying
experiment overrides? In this approach, defaults are always there and can be
optionally overridden.

[Ashok vardhan, 2015/08/27 00:30:46]

Done.

     return old_relevance;
+  }
 
   const base::TimeDelta time_since_last_visit =
       base::Time::Now() - match.url_info.last_visit();
@@ -463,6 +474,8 @@ HistoryURLProvider::HistoryURLProvider(AutocompleteProviderClient* client,
     : HistoryProvider(AutocompleteProvider::TYPE_HISTORY_URL, client),
       listener_(listener),
       params_(NULL) {
+  // Initialize the default HUP scoring params.
+  OmniboxFieldTrial::GetDefaultHUPScoringParams(&scoring_params_);
   // Initialize HUP scoring params based on the current experiment.
   OmniboxFieldTrial::GetExperimentalHUPScoringParams(&scoring_params_);
 }