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

Unified Diff: chrome/browser/task_profiler/task_profiler_data_serializer_unittest.cc

Issue 2881493004: Marshal 64 bit byte counts as double to the profiler. (Closed)
Patch Set: Fix the serialization unit test. Created 3 years, 7 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
Index: chrome/browser/task_profiler/task_profiler_data_serializer_unittest.cc
diff --git a/chrome/browser/task_profiler/task_profiler_data_serializer_unittest.cc b/chrome/browser/task_profiler/task_profiler_data_serializer_unittest.cc
index 5ff71d6b3f60819d62a124d1d7e83e7d431acc89..c97ff6b66bbe5150e1a07ba55f7ed7b7fa70795f 100644
--- a/chrome/browser/task_profiler/task_profiler_data_serializer_unittest.cc
+++ b/chrome/browser/task_profiler/task_profiler_data_serializer_unittest.cc
@@ -112,11 +112,11 @@ TEST(TaskProfilerDataSerializerTest, SerializeProcessDataToJson) {
"\"birth_thread\":\"CrBrowserMain\","
"\"death_data\":{"
"\"alloc_ops\":127,"
- "\"alloc_overhead_bytes\":201,"
- "\"allocated_bytes\":2013,"
+ "\"alloc_overhead_bytes\":201.0,"
+ "\"allocated_bytes\":2013.0,"
"\"count\":37,"
"\"free_ops\":120,"
- "\"freed_bytes\":1092,"
+ "\"freed_bytes\":1092.0,"
"\"max_allocated_bytes\":1500,"
"\"queue_ms\":79,"
"\"queue_ms_max\":53,"
@@ -135,11 +135,11 @@ TEST(TaskProfilerDataSerializerTest, SerializeProcessDataToJson) {
"\"birth_thread\":\"Chrome_IOThread\","
"\"death_data\":{"
"\"alloc_ops\":1207,"
- "\"alloc_overhead_bytes\":2001,"
- "\"allocated_bytes\":20013,"
+ "\"alloc_overhead_bytes\":2001.0,"
+ "\"allocated_bytes\":20013.0,"
"\"count\":41,"
"\"free_ops\":1200,"
- "\"freed_bytes\":10092,"
+ "\"freed_bytes\":10092.0,"
"\"max_allocated_bytes\":15000,"
"\"queue_ms\":2079,"
"\"queue_ms_max\":2053,"

Powered by Google App Engine
This is Rietveld 408576698