Launch HQP & HUP score changes.

components/omnibox/browser/omnibox_field_trial_unittest.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 "components/omnibox/browser/omnibox_field_trial.h"
 
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/metrics/field_trial.h"
@@ skipping 428 matching lines @@
             scoring_params.typed_count_buckets.buckets()[2]);
   EXPECT_EQ(11, scoring_params.visited_count_buckets.relevance_cap());
   EXPECT_EQ(31, scoring_params.visited_count_buckets.half_life_days());
   ASSERT_EQ(2u, scoring_params.visited_count_buckets.buckets().size());
   EXPECT_EQ(std::make_pair(5.0, 300),
             scoring_params.visited_count_buckets.buckets()[0]);
   EXPECT_EQ(std::make_pair(0.0, 200),
             scoring_params.visited_count_buckets.buckets()[1]);
 }
 
+TEST_F(OmniboxFieldTrialTest, HUPNewScoringFieldTrialWithDecayFactor) {
+  {
+    std::map<std::string, std::string> params;
+    params[std::string(OmniboxFieldTrial::kHUPNewScoringEnabledParam)] = "1";

[Peter Kasting, 2015/08/27 00:50:45]

Do the std::string() calls here need to be explicitly made?

[Ashok vardhan, 2015/08/27 19:09:27]

I think so, to convert const char to string. Followed the existing tests.

+    params[std::string(
+        OmniboxFieldTrial::kHUPNewScoringTypedCountHalfLifeTimeParam)] = "10";
+    params[std::string(
+        OmniboxFieldTrial::kHUPNewScoringTypedCountUseDecayFactorParam)] = "1";
+    params[std::string(
+        OmniboxFieldTrial::kHUPNewScoringTypedCountScoreBucketsParam)] =
+        "0.1:100,0.5:500,1.0:1000";
+    ASSERT_TRUE(variations::AssociateVariationParams(
+        OmniboxFieldTrial::kBundledExperimentFieldTrialName, "A", params));
+  }
+  base::FieldTrialList::CreateFieldTrial(
+      OmniboxFieldTrial::kBundledExperimentFieldTrialName, "A");
+
+  HUPScoringParams scoring_params;
+  OmniboxFieldTrial::GetExperimentalHUPScoringParams(&scoring_params);
+  EXPECT_TRUE(scoring_params.experimental_scoring_enabled);
+  EXPECT_EQ(10, scoring_params.typed_count_buckets.half_life_days());
+  ASSERT_EQ(3u, scoring_params.typed_count_buckets.buckets().size());
+  ASSERT_EQ(true, scoring_params.typed_count_buckets.use_decay_factor());
+}
+
 TEST_F(OmniboxFieldTrialTest, HalfLifeTimeDecay) {
   HUPScoringParams::ScoreBuckets buckets;
 
   // No decay by default.
   EXPECT_EQ(1.0, buckets.HalfLifeTimeDecay(base::TimeDelta::FromDays(7)));
 
   buckets.set_half_life_days(7);
   EXPECT_EQ(0.5, buckets.HalfLifeTimeDecay(base::TimeDelta::FromDays(7)));
   EXPECT_EQ(0.25, buckets.HalfLifeTimeDecay(base::TimeDelta::FromDays(14)));
   EXPECT_EQ(1.0, buckets.HalfLifeTimeDecay(base::TimeDelta::FromDays(0)));
@@ skipping 32 matching lines @@
       NULL, NULL, false,
       OmniboxFieldTrial::kDefaultMinimumTimeBetweenSuggestQueriesMs);
 
   // Valid params.
   VerifySuggestPollingStrategy("true", "50", true, 50);
   VerifySuggestPollingStrategy(NULL, "35", false, 35);
   VerifySuggestPollingStrategy(
       "true", NULL, true,
       OmniboxFieldTrial::kDefaultMinimumTimeBetweenSuggestQueriesMs);
 }