New smart passsword bubble algorithm.

chrome/browser/ui/passwords/password_bubble_experiment.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_PASSWORDS_PASSWORD_BUBBLE_EXPERIMENT_H_
 #define CHROME_BROWSER_UI_PASSWORDS_PASSWORD_BUBBLE_EXPERIMENT_H_
 
 #include "base/macros.h"
 #include "components/password_manager/core/browser/password_manager_metrics_util.h"
 
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 class PrefService;
 
 // These functions handle the algorithms according to which the "Save password?"
-// bubble is shown to user.
+// bubble shows "No thanks" or "Never for this site" button default.
 namespace password_bubble_experiment {
 
 void RegisterPrefs(user_prefs::PrefRegistrySyncable* registry);
 
-// The decision is made based on the "PasswordBubbleAlgorithm" finch experiment.
-// The default value is true.
+// Returns true if "Never for this site" should be the default negative option.
+// Otherwise it's "No thanks". The default value is false.
 // It should be called before showing the "Save Password?" dialog.
-bool ShouldShowBubble(PrefService* prefs);
+bool ShouldShowNeverForThisSiteDefault(PrefService* prefs);
 
 // Should be called when user dismisses the "Save Password?" dialog. It stores
 // the statistics about interactions with the bubble.
 void RecordBubbleClosed(
     PrefService* prefs,
     password_manager::metrics_util::UIDismissalReason reason);
 
 // The name of the finch experiment controlling the algorithm.
 extern const char kExperimentName[];
 
-// The group name for the time based algorithm.
-extern const char kGroupTimeSpanBased[];
-
-// The group name for the probability algorithm.
-extern const char kGroupProbabilityBased[];
-
-// For "Probability" group. The additional "Saves" to be added to the model.
-extern const char kParamProbabilityFakeSaves[];
-
-// For "Probability" group. The interaction history length.
-extern const char kParamProbabilityInteractionsCount[];
-
-// For "TimeSpan" group. The time span until the nope counter is zeroed.
-extern const char kParamTimeSpan[];
-
-// For "TimeSpan" group. The nopes threshold.
-extern const char kParamTimeSpanNopeThreshold[];
+// The consecutive nopes threshold after which the user sees "Never for this

[vabr (Chromium), 2015/03/24 09:09:04]

nit: The comment sounds like the constant below is the (numerical) threshold,
but it is only the name of that parameter. Maybe specify, that this can be used
by GetVariationParamValue to get the actual value, or say something similar?

[vasilii, 2015/03/24 10:30:53]

Done.

+// site" button by default.
+extern const char kParamNopeThreshold[];
 
 }  // namespace password_bubble_experiment
 
 #endif  // CHROME_BROWSER_UI_PASSWORDS_PASSWORD_BUBBLE_EXPERIMENT_H_