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

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

Issue 1104053004: Cleanup: removing unused descendants information from tracked objects (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: isherman@ comments Created 5 years, 8 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.cc
diff --git a/chrome/browser/task_profiler/task_profiler_data_serializer.cc b/chrome/browser/task_profiler/task_profiler_data_serializer.cc
index d072d657f555af94881074374d4cda680e48d492..c0f49721bf3a9982d067511b3b54f3569599eee9 100644
--- a/chrome/browser/task_profiler/task_profiler_data_serializer.cc
+++ b/chrome/browser/task_profiler/task_profiler_data_serializer.cc
@@ -19,7 +19,6 @@ using base::Value;
using tracked_objects::BirthOnThreadSnapshot;
using tracked_objects::DeathDataSnapshot;
using tracked_objects::LocationSnapshot;
-using tracked_objects::ParentChildPairSnapshot;
using tracked_objects::TaskSnapshot;
using tracked_objects::ProcessDataPhaseSnapshot;
@@ -95,15 +94,6 @@ void TaskProfilerDataSerializer::ToValue(
dictionary->SetInteger("process_id", process_id);
dictionary->SetString("process_type",
content::GetProcessTypeNameInEnglish(process_type));
-
- scoped_ptr<base::ListValue> descendants_list(new base::ListValue);
- for (const auto& entry : process_data_phase.descendants) {
- scoped_ptr<base::DictionaryValue> parent_child(new base::DictionaryValue);
- BirthOnThreadSnapshotToValue(entry.parent, "parent", parent_child.get());
- BirthOnThreadSnapshotToValue(entry.child, "child", parent_child.get());
- descendants_list->Append(parent_child.release());
- }
- dictionary->Set("descendants", descendants_list.release());
}
} // namespace task_profiler
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/task_profiler/task_profiler_data_serializer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698