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

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

Issue 2804693002: Remove coordinates from LatencyInfo objects. (Closed)
Patch Set: Fix mojo latencyInfo test Created 3 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
« no previous file with comments | « tools/ipc_fuzzer/fuzzer/fuzzer.cc ('k') | ui/latency/ipc/latency_info_param_traits_unittest.cc » ('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 23a4eede91bceeee4d985dc6901129d3ace17d60..d1e6f509ea957124bd82b1aa443b2a574d649e5d 100644
--- a/ui/latency/ipc/latency_info_param_traits.cc
+++ b/ui/latency/ipc/latency_info_param_traits.cc
@@ -42,10 +42,6 @@ void ParamTraits<ui::LatencyInfo>::GetSize(base::PickleSizer* s,
const param_type& p) {
GetParamSize(s, p.trace_name_);
GetParamSize(s, p.latency_components_);
- GetParamSize(s, p.input_coordinates_size_);
- for (size_t i = 0; i < p.input_coordinates_size_; i++) {
- GetParamSize(s, p.input_coordinates_[i]);
- }
GetParamSize(s, p.trace_id_);
GetParamSize(s, p.terminated_);
GetParamSize(s, p.source_event_type_);
@@ -54,10 +50,6 @@ void ParamTraits<ui::LatencyInfo>::GetSize(base::PickleSizer* s,
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.input_coordinates_size_);
- for (size_t i = 0; i < p.input_coordinates_size_; i++) {
- WriteParam(m, p.input_coordinates_[i]);
- }
WriteParam(m, p.trace_id_);
WriteParam(m, p.terminated_);
WriteParam(m, p.source_event_type_);
@@ -71,17 +63,6 @@ bool ParamTraits<ui::LatencyInfo>::Read(const base::Pickle* m,
if (!ReadParam(m, iter, &p->latency_components_))
return false;
- gfx::PointF input_coordinates;
- uint32_t input_coordinates_size;
- if (!ReadParam(m, iter, &input_coordinates_size))
- return false;
- for (size_t i = 0; i < input_coordinates_size; i++) {
- if (!ReadParam(m, iter, &input_coordinates))
- return false;
- if (!p->AddInputCoordinate(input_coordinates))
- return false;
- }
-
if (!ReadParam(m, iter, &p->trace_id_))
return false;
if (!ReadParam(m, iter, &p->terminated_))
@@ -97,12 +78,6 @@ void ParamTraits<ui::LatencyInfo>::Log(const param_type& p, std::string* l) {
l->append(" ");
LogParam(p.latency_components_, l);
l->append(" ");
- LogParam(p.input_coordinates_size_, l);
- l->append(" ");
- for (size_t i = 0; i < p.input_coordinates_size_; i++) {
- LogParam(p.input_coordinates_[i], l);
- l->append(" ");
- }
LogParam(p.trace_id_, l);
l->append(" ");
LogParam(p.terminated_, l);
« no previous file with comments | « tools/ipc_fuzzer/fuzzer/fuzzer.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