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

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

Issue 2954473002: Record accuracy of expected queueing time metric. (Closed)
Patch Set: Set trace ID in test. 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
« no previous file with comments | « tools/metrics/histograms/histograms.xml ('k') | ui/latency/latency_info.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 89dd8b31e3c6f51c1859f65a8ebf4756fa712473..87b9a8907d35440617f7d15ca6177d7d3d663346 100644
--- a/ui/latency/ipc/latency_info_param_traits.cc
+++ b/ui/latency/ipc/latency_info_param_traits.cc
@@ -46,6 +46,7 @@ void ParamTraits<ui::LatencyInfo>::GetSize(base::PickleSizer* s,
GetParamSize(s, p.began_);
GetParamSize(s, p.terminated_);
GetParamSize(s, p.source_event_type_);
+ GetParamSize(s, p.expected_queueing_time_on_dispatch_);
}
void ParamTraits<ui::LatencyInfo>::Write(base::Pickle* m, const param_type& p) {
@@ -55,6 +56,7 @@ void ParamTraits<ui::LatencyInfo>::Write(base::Pickle* m, const param_type& p) {
WriteParam(m, p.began_);
WriteParam(m, p.terminated_);
WriteParam(m, p.source_event_type_);
+ WriteParam(m, p.expected_queueing_time_on_dispatch_);
}
bool ParamTraits<ui::LatencyInfo>::Read(const base::Pickle* m,
@@ -73,6 +75,8 @@ bool ParamTraits<ui::LatencyInfo>::Read(const base::Pickle* m,
return false;
if (!ReadParam(m, iter, &p->source_event_type_))
return false;
+ if (!ReadParam(m, iter, &p->expected_queueing_time_on_dispatch_))
+ return false;
return true;
}
@@ -89,6 +93,8 @@ void ParamTraits<ui::LatencyInfo>::Log(const param_type& p, std::string* l) {
LogParam(p.terminated_, l);
l->append(" ");
LogParam(p.source_event_type_, l);
+ l->append(" ");
+ LogParam(p.expected_queueing_time_on_dispatch_, l);
}
} // namespace IPC
« no previous file with comments | « tools/metrics/histograms/histograms.xml ('k') | ui/latency/latency_info.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698