| Index: cc/surfaces/compositor_frame_sink_support_unittest.cc
|
| diff --git a/cc/surfaces/compositor_frame_sink_support_unittest.cc b/cc/surfaces/compositor_frame_sink_support_unittest.cc
|
| index 319471a9296c4d5b16daf5527727ca74d117b386..8c87647721434237bd33cae7abf4a76c8985bcaa 100644
|
| --- a/cc/surfaces/compositor_frame_sink_support_unittest.cc
|
| +++ b/cc/surfaces/compositor_frame_sink_support_unittest.cc
|
| @@ -826,12 +826,20 @@ TEST_F(CompositorFrameSinkSupportTest,
|
|
|
| ui::LatencyInfo aggregated_latency_info = info_list[0];
|
| aggregated_latency_info.AddNewLatencyFrom(info_list[1]);
|
| - EXPECT_EQ(2u, aggregated_latency_info.latency_components().size());
|
| +
|
| + // Two components are the original ones, and the third one is
|
| + // DISPLAY_COMPOSITOR_RECEIVED_FRAME_COMPONENT, logged on compositor frame
|
| + // submit.
|
| + EXPECT_EQ(3u, aggregated_latency_info.latency_components().size());
|
|
|
| ui::LatencyInfo::LatencyComponent comp1;
|
| EXPECT_TRUE(
|
| aggregated_latency_info.FindLatency(latency_type1, latency_id1, &comp1));
|
| EXPECT_EQ(latency_sequence_number1, comp1.sequence_number);
|
| + EXPECT_TRUE(
|
| + aggregated_latency_info.FindLatency(latency_type2, latency_id2, nullptr));
|
| + EXPECT_TRUE(aggregated_latency_info.FindLatency(
|
| + ui::DISPLAY_COMPOSITOR_RECEIVED_FRAME_COMPONENT, nullptr));
|
| }
|
|
|
| // Checks whether the latency info are moved to the new surface from the old
|
| @@ -894,12 +902,20 @@ TEST_F(CompositorFrameSinkSupportTest,
|
|
|
| ui::LatencyInfo aggregated_latency_info = info_list[0];
|
| aggregated_latency_info.AddNewLatencyFrom(info_list[1]);
|
| - EXPECT_EQ(2u, aggregated_latency_info.latency_components().size());
|
| +
|
| + // Two components are the original ones, and the third one is
|
| + // DISPLAY_COMPOSITOR_RECEIVED_FRAME_COMPONENT, logged on compositor frame
|
| + // submit.
|
| + EXPECT_EQ(3u, aggregated_latency_info.latency_components().size());
|
|
|
| ui::LatencyInfo::LatencyComponent comp1;
|
| EXPECT_TRUE(
|
| aggregated_latency_info.FindLatency(latency_type1, latency_id1, &comp1));
|
| EXPECT_EQ(latency_sequence_number1, comp1.sequence_number);
|
| + EXPECT_TRUE(
|
| + aggregated_latency_info.FindLatency(latency_type2, latency_id2, nullptr));
|
| + EXPECT_TRUE(aggregated_latency_info.FindLatency(
|
| + ui::DISPLAY_COMPOSITOR_RECEIVED_FRAME_COMPONENT, nullptr));
|
| }
|
|
|
| // Checks whether the latency info are moved to the new surface from the old
|
| @@ -965,12 +981,20 @@ TEST_F(CompositorFrameSinkSupportTest,
|
|
|
| ui::LatencyInfo aggregated_latency_info = info_list[0];
|
| aggregated_latency_info.AddNewLatencyFrom(info_list[1]);
|
| - EXPECT_EQ(2u, aggregated_latency_info.latency_components().size());
|
| +
|
| + // Two components are the original ones, and the third one is
|
| + // DISPLAY_COMPOSITOR_RECEIVED_FRAME_COMPONENT, logged on compositor frame
|
| + // submit.
|
| + EXPECT_EQ(3u, aggregated_latency_info.latency_components().size());
|
|
|
| ui::LatencyInfo::LatencyComponent comp1;
|
| EXPECT_TRUE(
|
| aggregated_latency_info.FindLatency(latency_type1, latency_id1, &comp1));
|
| EXPECT_EQ(latency_sequence_number1, comp1.sequence_number);
|
| + EXPECT_TRUE(
|
| + aggregated_latency_info.FindLatency(latency_type2, latency_id2, nullptr));
|
| + EXPECT_TRUE(aggregated_latency_info.FindLatency(
|
| + ui::DISPLAY_COMPOSITOR_RECEIVED_FRAME_COMPONENT, nullptr));
|
| }
|
|
|
| TEST_F(CompositorFrameSinkSupportTest, PassesOnBeginFrameAcks) {
|
|
|