Index: chrome/browser/prefs/pref_metrics_service.cc |
diff --git a/chrome/browser/prefs/pref_metrics_service.cc b/chrome/browser/prefs/pref_metrics_service.cc |
index 8b6f389f961c1d5cf0a3c24a9865c05651e721a3..0030632f86c31d636f137b11ff0d0501f56d7a79 100644 |
--- a/chrome/browser/prefs/pref_metrics_service.cc |
+++ b/chrome/browser/prefs/pref_metrics_service.cc |
@@ -22,6 +22,7 @@ |
#include "chrome/common/chrome_switches.h" |
#include "chrome/common/pref_names.h" |
#include "components/browser_context_keyed_service/browser_context_dependency_manager.h" |
+#include "components/rappor/rappor_service.h" |
#include "crypto/hmac.h" |
#include "net/base/registry_controlled_domains/registry_controlled_domain.h" |
@@ -29,6 +30,15 @@ namespace { |
const int kSessionStartupPrefValueMax = SessionStartupPref::kPrefValueMax; |
+const rappor::RapporParameters kHomePageRapporParameters = { |
+ "Settings.HomePage", 16 /* Bloom filter size bytes */, |
Ilya Sherman
2014/01/10 11:00:32
Optional nit: One arg per line might be a little c
Steven Holte
2014/01/14 00:47:54
Done.
|
+ 4 /* Bloom filter hash count */, |
+ rappor::PROBABILITY_75 /* Fake data probability */, |
+ rappor::PROBABILITY_50 /* Fake one probability */, |
+ rappor::PROBABILITY_75 /* One coin probability */, |
+ rappor::PROBABILITY_50 /* Zero coin probability */ |
Ilya Sherman
2014/01/10 11:00:32
This is a lot of manual configuration. Do all of
Steven Holte
2014/01/14 00:47:54
We should have defaults for this eventually, but w
Alexei Svitkine (slow)
2014/01/22 22:13:28
This is one area that still concerns me. If the AP
|
+}; |
+ |
} // namespace |
PrefMetricsService::PrefMetricsService(Profile* profile) |
@@ -78,6 +88,8 @@ void PrefMetricsService::RecordLaunchPrefs() { |
"Settings.HomePageEngineType", |
TemplateURLPrepopulateData::GetEngineType(homepage_url), |
SEARCH_ENGINE_MAX); |
+ g_browser_process->rappor_service()->RecordUrl( |
+ kHomePageRapporParameters, homepage_url); |
} |
} |