Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(813)

Unified Diff: components/metrics/metrics_log_manager_unittest.cc

Issue 2358223002: Add log date to the metrics log (Closed)
Patch Set: add deprectaed comments Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/metrics/metrics_log_manager.cc ('k') | components/metrics/metrics_pref_names.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/metrics/metrics_log_manager_unittest.cc
diff --git a/components/metrics/metrics_log_manager_unittest.cc b/components/metrics/metrics_log_manager_unittest.cc
index 48f95f8ef9c8cd616335f74814492c6fc25b0436..fbd1dd5d2b5e4e0ee6f90e56d6793a7f292c0729 100644
--- a/components/metrics/metrics_log_manager_unittest.cc
+++ b/components/metrics/metrics_log_manager_unittest.cc
@@ -37,7 +37,7 @@ class TestLogPrefService : public TestingPrefServiceSimple {
list_length = GetList(prefs::kMetricsInitialLogs)->GetSize();
else
list_length = GetList(prefs::kMetricsOngoingLogs)->GetSize();
- return list_length / 2;
+ return list_length;
}
};
@@ -163,8 +163,8 @@ TEST(MetricsLogManagerTest, StoreAndLoad) {
// Simulate a log having already been unsent from a previous session.
{
std::string log("proto");
- PersistedLogs ongoing_logs(&pref_service, prefs::kMetricsOngoingLogs, 1,
- 1, 0);
+ PersistedLogs ongoing_logs(&pref_service, prefs::kMetricsOngoingLogs,
+ prefs::kDeprecatedMetricsOngoingLogs, 1, 1, 0);
ongoing_logs.StoreLog(log);
ongoing_logs.SerializeLogs();
}
« no previous file with comments | « components/metrics/metrics_log_manager.cc ('k') | components/metrics/metrics_pref_names.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698