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

Unified Diff: ui/latency/mojo/latency_info_struct_traits.cc

Issue 2954473002: Record accuracy of expected queueing time metric. (Closed)
Patch Set: Fix include. 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/mojo/latency_info_struct_traits.cc
diff --git a/ui/latency/mojo/latency_info_struct_traits.cc b/ui/latency/mojo/latency_info_struct_traits.cc
index 12daf3348620a48e7217aaaebb8a53d61b811828..c9b3b59b4b66ba32204ab13e0f25b0a1f3bc96ae 100644
--- a/ui/latency/mojo/latency_info_struct_traits.cc
+++ b/ui/latency/mojo/latency_info_struct_traits.cc
@@ -303,6 +303,12 @@ StructTraits<ui::mojom::LatencyInfoDataView,
return UISourceEventTypeToMojo(info.source_event_type());
}
+// static
+base::TimeDelta StructTraits<ui::mojom::LatencyInfoDataView, ui::LatencyInfo>::
+ expected_queueing_time_on_dispatch(const ui::LatencyInfo& info) {
+ return info.expected_queueing_time_on_dispatch();
+}
+
// static
bool StructTraits<ui::mojom::LatencyInfoDataView, ui::LatencyInfo>::Read(
ui::mojom::LatencyInfoDataView data,
@@ -327,7 +333,9 @@ bool StructTraits<ui::mojom::LatencyInfoDataView, ui::LatencyInfo>::Read(
out->coalesced_ = data.coalesced();
out->terminated_ = data.terminated();
out->source_event_type_ = MojoSourceEventTypeToUI(data.source_event_type());
- return true;
+
+ return data.ReadExpectedQueueingTimeOnDispatch(
+ &out->expected_queueing_time_on_dispatch_);
}
} // namespace mojo
« tools/metrics/histograms/histograms.xml ('K') | « ui/latency/mojo/latency_info_struct_traits.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698