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

Unified Diff: ui/latency/ipc/latency_info_param_traits.cc

Issue 2953073002: LatencyInfo trace_id_ no longer dependent on sequence_number. (Closed)
Patch Set: Address nit. Created 3 years, 6 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: ui/latency/ipc/latency_info_param_traits.cc
diff --git a/ui/latency/ipc/latency_info_param_traits.cc b/ui/latency/ipc/latency_info_param_traits.cc
index d1e6f509ea957124bd82b1aa443b2a574d649e5d..89dd8b31e3c6f51c1859f65a8ebf4756fa712473 100644
--- a/ui/latency/ipc/latency_info_param_traits.cc
+++ b/ui/latency/ipc/latency_info_param_traits.cc
@@ -43,6 +43,7 @@ void ParamTraits<ui::LatencyInfo>::GetSize(base::PickleSizer* s,
GetParamSize(s, p.trace_name_);
GetParamSize(s, p.latency_components_);
GetParamSize(s, p.trace_id_);
+ GetParamSize(s, p.began_);
GetParamSize(s, p.terminated_);
GetParamSize(s, p.source_event_type_);
}
@@ -51,6 +52,7 @@ void ParamTraits<ui::LatencyInfo>::Write(base::Pickle* m, const param_type& p) {
WriteParam(m, p.trace_name_);
WriteParam(m, p.latency_components_);
WriteParam(m, p.trace_id_);
+ WriteParam(m, p.began_);
WriteParam(m, p.terminated_);
WriteParam(m, p.source_event_type_);
}
@@ -65,6 +67,8 @@ bool ParamTraits<ui::LatencyInfo>::Read(const base::Pickle* m,
if (!ReadParam(m, iter, &p->trace_id_))
return false;
+ if (!ReadParam(m, iter, &p->began_))
+ return false;
if (!ReadParam(m, iter, &p->terminated_))
return false;
if (!ReadParam(m, iter, &p->source_event_type_))
@@ -80,6 +84,8 @@ void ParamTraits<ui::LatencyInfo>::Log(const param_type& p, std::string* l) {
l->append(" ");
LogParam(p.trace_id_, l);
l->append(" ");
+ LogParam(p.began_, l);
+ l->append(" ");
LogParam(p.terminated_, l);
l->append(" ");
LogParam(p.source_event_type_, l);
« no previous file with comments | « content/common/input/input_param_traits_unittest.cc ('k') | ui/latency/ipc/latency_info_param_traits_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698