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

Unified Diff: content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc

Issue 2953073002: LatencyInfo trace_id_ no longer dependent on sequence_number. (Closed)
Patch Set: Fix cc_unittests 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: content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
diff --git a/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc b/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
index 743b0619c1ff9f090074b6d1d2673582ea28c869..78d001ca1c5e4dd157d1690815d0c76b0330829a 100644
--- a/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
+++ b/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
@@ -813,6 +813,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, TouchBlockingAndQueueingTime) {
tracker()->OnInputEvent(event, &latency);
ui::LatencyInfo fake_latency;
+ fake_latency.set_trace_id(5);
brianderson 2017/06/27 18:00:58 Would it make sense to have 5 be a const variable
tdresser 2017/06/27 18:33:12 Done.
fake_latency.set_source_event_type(ui::SourceEventType::TOUCH);
fake_latency.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT,
@@ -859,6 +860,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, TouchBlockingAndQueueingTime) {
EXPECT_EQ(2U, latency.latency_components().size());
ui::LatencyInfo fake_latency;
+ fake_latency.set_trace_id(5);
fake_latency.set_source_event_type(ui::SourceEventType::TOUCH);
fake_latency.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT,
@@ -902,6 +904,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, TouchBlockingAndQueueingTime) {
EXPECT_EQ(2U, latency.latency_components().size());
ui::LatencyInfo fake_latency;
+ fake_latency.set_trace_id(5);
fake_latency.set_source_event_type(ui::SourceEventType::TOUCH);
fake_latency.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT,
@@ -1004,6 +1007,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, KeyBlockingAndQueueingTime) {
tracker()->OnInputEvent(event, &latency_info);
ui::LatencyInfo fake_latency;
+ fake_latency.set_trace_id(5);
fake_latency.set_source_event_type(ui::SourceEventType::KEY_PRESS);
fake_latency.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT,
@@ -1060,6 +1064,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, KeyUILatency) {
blink::WebInputEvent::kNoModifiers,
base::TimeTicks::Now());
ui::LatencyInfo latency_info;
+ latency_info.set_trace_id(5);
latency_info.set_source_event_type(ui::SourceEventType::KEY_PRESS);
latency_info.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_UI_COMPONENT, 0, 0,
@@ -1092,6 +1097,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, KeyAckedLatency) {
blink::WebInputEvent::kNoModifiers,
base::TimeTicks::Now());
ui::LatencyInfo latency_info;
+ latency_info.set_trace_id(5);
latency_info.set_source_event_type(ui::SourceEventType::KEY_PRESS);
latency_info.AddLatencyNumberWithTimestamp(
@@ -1126,6 +1132,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, KeyEndToEndLatency) {
int event_timestamps_microseconds[] = {11, 24};
ui::LatencyInfo latency_info;
+ latency_info.set_trace_id(5);
latency_info.set_source_event_type(ui::SourceEventType::KEY_PRESS);
latency_info.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT, 0, 0,
@@ -1185,6 +1192,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest,
tracker()->OnInputEvent(event, &latency);
ui::LatencyInfo fake_latency;
+ fake_latency.set_trace_id(5);
fake_latency.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT,
tracker()->latency_component_id(), 0,
@@ -1254,6 +1262,7 @@ TEST_F(RenderWidgetHostLatencyTrackerTest, WheelDuringMultiFingerTouch) {
tracker()->OnInputEvent(touch_event, &latency);
ui::LatencyInfo fake_latency;
+ fake_latency.set_trace_id(5);
fake_latency.set_source_event_type(ui::SourceEventType::TOUCH);
fake_latency.AddLatencyNumberWithTimestamp(
ui::INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT,

Powered by Google App Engine
This is Rietveld 408576698