Chromium Code Reviews| Index: chrome/browser/engagement/site_engagement_service.cc |
| diff --git a/chrome/browser/engagement/site_engagement_service.cc b/chrome/browser/engagement/site_engagement_service.cc |
| index ba0e74dd54de955f2aa1a583b897e198d336ad7b..7ac64f109c3f128aeb745a24653acae811a9cdbe 100644 |
| --- a/chrome/browser/engagement/site_engagement_service.cc |
| +++ b/chrome/browser/engagement/site_engagement_service.cc |
| @@ -126,6 +126,7 @@ double SiteEngagementScore::param_values[] = { |
| 0.01, // HIDDEN_MEDIA_POINTS |
| 5, // WEB_APP_INSTALLED_POINTS |
| 0.5, // FIRST_DAILY_ENGAGEMENT |
| + 30, // BOOTSTRAP_POINTS |
|
dominickn
2016/02/04 03:11:24
Nit: as discussed offline wrt UMA stats, change th
calamity
2016/02/04 23:09:09
Done.
|
| }; |
| const char* SiteEngagementScore::kRawScoreKey = "rawScore"; |
| @@ -170,6 +171,10 @@ double SiteEngagementScore::GetFirstDailyEngagementPoints() { |
| return param_values[FIRST_DAILY_ENGAGEMENT]; |
| } |
| +double SiteEngagementScore::GetBootstrapPoints() { |
| + return param_values[BOOTSTRAP_POINTS]; |
| +} |
| + |
| void SiteEngagementScore::UpdateFromVariations() { |
| double param_vals[MAX_VARIATION]; |
| @@ -334,6 +339,7 @@ void SiteEngagementScore::SetParamValuesForTesting() { |
| param_values[VISIBLE_MEDIA_POINTS] = 0.02; |
| param_values[HIDDEN_MEDIA_POINTS] = 0.01; |
| param_values[WEB_APP_INSTALLED_POINTS] = 5; |
| + param_values[BOOTSTRAP_POINTS] = 30; |
| // This is set to zero to avoid interference with tests and is set when |
| // testing this functionality. |
| @@ -519,6 +525,11 @@ std::map<GURL, double> SiteEngagementService::GetScoreMap() { |
| return score_map; |
| } |
| +bool SiteEngagementService::IsBootstrapped() { |
| + return GetTotalEngagementPoints() >= |
| + SiteEngagementScore::GetBootstrapPoints(); |
| +} |
| + |
| SiteEngagementService::SiteEngagementService(Profile* profile, |
| scoped_ptr<base::Clock> clock) |
| : profile_(profile), clock_(std::move(clock)), weak_factory_(this) { |