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

Unified Diff: components/metrics/metrics_log_manager_unittest.cc

Issue 2588873002: Delegate PersistedLogs metrics recording (Closed)
Patch Set: Fix tests Created 4 years 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/persisted_logs.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 fbd1dd5d2b5e4e0ee6f90e56d6793a7f292c0729..8040ed083eca265aa85072c1c8b4e4b952ed100d 100644
--- a/components/metrics/metrics_log_manager_unittest.cc
+++ b/components/metrics/metrics_log_manager_unittest.cc
@@ -13,6 +13,7 @@
#include "base/memory/ptr_util.h"
#include "components/metrics/metrics_log.h"
#include "components/metrics/metrics_pref_names.h"
+#include "components/metrics/persisted_logs_metrics_impl.h"
#include "components/metrics/test_metrics_service_client.h"
#include "components/prefs/pref_registry_simple.h"
#include "components/prefs/testing_pref_service.h"
@@ -163,7 +164,9 @@ 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,
+ PersistedLogs ongoing_logs(std::unique_ptr<PersistedLogsMetricsImpl>(
+ new PersistedLogsMetricsImpl()),
+ &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/persisted_logs.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698