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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/metrics/metrics_log_serializer_unittest.cc

Issue 26646003: MetricsService: Send a hash of the UMA log in a header. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 2 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 | « chrome/browser/metrics/metrics_log_serializer.cc ('k') | chrome/browser/metrics/metrics_service.cc » ('j') | chrome/common/metrics/metrics_log_manager.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/metrics_log_serializer_unittest.cc
===================================================================
--- chrome/browser/metrics/metrics_log_serializer_unittest.cc (revision 228712)
+++ chrome/browser/metrics/metrics_log_serializer_unittest.cc (working copy)
@@ -21,7 +21,7 @@
// Store and retrieve empty list.
TEST(MetricsLogSerializerTest, EmptyLogList) {
ListValue list;
- std::vector<std::string> local_list;
+ std::vector<MetricsLogManager::SerializedLog> local_list;
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
@@ -38,8 +38,8 @@
TEST(MetricsLogSerializerTest, SingleElementLogList) {
ListValue list;
- std::vector<std::string> local_list(1);
- local_list[0] = "Hello world!";
+ std::vector<MetricsLogManager::SerializedLog> local_list(1);
+ local_list[0].SetLogText("Hello world!");
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
@@ -81,20 +81,19 @@
ListValue list;
size_t log_count = kListLengthLimit * 5;
- std::vector<std::string> local_list(log_count);
- for (size_t i = 0; i < local_list.size(); ++i) {
- local_list[0] = "x";
- }
+ std::vector<MetricsLogManager::SerializedLog> local_list(log_count);
+ for (size_t i = 0; i < local_list.size(); ++i)
+ local_list[i].SetLogText("x");
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
- std::vector<std::string> result_list;
+ std::vector<MetricsLogManager::SerializedLog> result_list;
EXPECT_EQ(
MetricsLogSerializer::RECALL_SUCCESS,
MetricsLogSerializer::ReadLogsFromPrefList(list, &result_list));
EXPECT_EQ(local_list.size(), result_list.size());
- EXPECT_TRUE(result_list.front().find("x") == 0);
+ EXPECT_TRUE(result_list.front().log_text().find("x") == 0);
}
// Store a set of logs over the length limit, but that doesn't reach the minimum
@@ -106,26 +105,28 @@
// Make log_count logs each slightly larger than
// kLogByteLimit / (log_count - 2)
// so that the minimum is reached before the oldest (first) two logs.
- std::vector<std::string> local_list(log_count);
+ std::vector<MetricsLogManager::SerializedLog> local_list(log_count);
size_t log_size = (kLogByteLimit / (log_count - 2)) + 2;
- local_list[0] = "one";
- local_list[1] = "two";
- local_list[2] = "three";
- local_list[log_count - 1] = "last";
+ local_list[0].SetLogText("one");
+ local_list[1].SetLogText("two");
+ local_list[2].SetLogText("three");
+ local_list[log_count - 1].SetLogText("last");
for (size_t i = 0; i < local_list.size(); ++i) {
- local_list[i].resize(log_size, ' ');
+ std::string log_text = local_list[i].log_text();
+ log_text.resize(log_size, ' ');
+ local_list[i].SetLogText(log_text);
}
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
- std::vector<std::string> result_list;
+ std::vector<MetricsLogManager::SerializedLog> result_list;
EXPECT_EQ(
MetricsLogSerializer::RECALL_SUCCESS,
MetricsLogSerializer::ReadLogsFromPrefList(list, &result_list));
EXPECT_EQ(local_list.size() - 2, result_list.size());
- EXPECT_TRUE(result_list.front().find("three") == 0);
- EXPECT_TRUE(result_list.back().find("last") == 0);
+ EXPECT_TRUE(result_list.front().log_text().find("three") == 0);
+ EXPECT_TRUE(result_list.back().log_text().find("last") == 0);
}
// Store a set of logs within the length limit, but well over the minimum
@@ -133,16 +134,18 @@
TEST(MetricsLogSerializerTest, ShortButLargeLogList) {
ListValue list;
- std::vector<std::string> local_list(kListLengthLimit);
+ std::vector<MetricsLogManager::SerializedLog> local_list(kListLengthLimit);
// Make the total byte count about twice the minimum.
size_t log_size = (kLogByteLimit / local_list.size()) * 2;
for (size_t i = 0; i < local_list.size(); ++i) {
- local_list[i].resize(log_size, ' ');
+ std::string log_text = local_list[i].log_text();
+ log_text.resize(log_size, ' ');
+ local_list[i].SetLogText(log_text);
}
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
- std::vector<std::string> result_list;
+ std::vector<MetricsLogManager::SerializedLog> result_list;
EXPECT_EQ(
MetricsLogSerializer::RECALL_SUCCESS,
MetricsLogSerializer::ReadLogsFromPrefList(list, &result_list));
@@ -155,31 +158,34 @@
ListValue list;
// Include twice the max number of logs.
- std::vector<std::string> local_list(kListLengthLimit * 2);
+ std::vector<MetricsLogManager::SerializedLog>
+ local_list(kListLengthLimit * 2);
// Make the total byte count about four times the minimum.
size_t log_size = (kLogByteLimit / local_list.size()) * 4;
- local_list[local_list.size() - kListLengthLimit] = "First to keep";
+ local_list[local_list.size() - kListLengthLimit].SetLogText("First to keep");
for (size_t i = 0; i < local_list.size(); ++i) {
- local_list[i].resize(log_size, ' ');
+ std::string log_text = local_list[i].log_text();
+ log_text.resize(log_size, ' ');
+ local_list[i].SetLogText(log_text);
}
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
- std::vector<std::string> result_list;
+ std::vector<MetricsLogManager::SerializedLog> result_list;
EXPECT_EQ(
MetricsLogSerializer::RECALL_SUCCESS,
MetricsLogSerializer::ReadLogsFromPrefList(list, &result_list));
// The max length should control the resulting size.
EXPECT_EQ(kListLengthLimit, result_list.size());
- EXPECT_TRUE(result_list.front().find("First to keep") == 0);
+ EXPECT_TRUE(result_list.front().log_text().find("First to keep") == 0);
}
// Induce LIST_SIZE_TOO_SMALL corruption
TEST(MetricsLogSerializerTest, SmallRecoveredListSize) {
ListValue list;
- std::vector<std::string> local_list(1);
- local_list[0] = "Hello world!";
+ std::vector<MetricsLogManager::SerializedLog> local_list(1);
+ local_list[0].SetLogText("Hello world!");
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
@@ -199,9 +205,9 @@
TEST(MetricsLogSerializerTest, RemoveSizeFromLogList) {
ListValue list;
- std::vector<std::string> local_list(2);
- local_list[0] = "one";
- local_list[1] = "two";
+ std::vector<MetricsLogManager::SerializedLog> local_list(2);
+ local_list[0].SetLogText("one");
+ local_list[1].SetLogText("two");
EXPECT_EQ(2U, local_list.size());
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
@@ -220,8 +226,8 @@
TEST(MetricsLogSerializerTest, CorruptSizeOfLogList) {
ListValue list;
- std::vector<std::string> local_list(1);
- local_list[0] = "Hello world!";
+ std::vector<MetricsLogManager::SerializedLog> local_list(1);
+ local_list[0].SetLogText("Hello world!");
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
@@ -241,8 +247,8 @@
TEST(MetricsLogSerializerTest, CorruptChecksumOfLogList) {
ListValue list;
- std::vector<std::string> local_list(1);
- local_list[0] = "Hello world!";
+ std::vector<MetricsLogManager::SerializedLog> local_list(1);
+ local_list[0].SetLogText("Hello world!");
MetricsLogSerializer::WriteLogsToPrefList(local_list, kListLengthLimit,
kLogByteLimit, &list);
« no previous file with comments | « chrome/browser/metrics/metrics_log_serializer.cc ('k') | chrome/browser/metrics/metrics_service.cc » ('j') | chrome/common/metrics/metrics_log_manager.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698