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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/surfaces/surface_synchronization_unittest.cc

Issue 2914023002: Remove LatencyInfo::sequence_number. (May break metrics).
Patch Set: Fix Windows. 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 | « cc/surfaces/compositor_frame_sink_support.cc ('k') | cc/trees/latency_info_swap_promise_monitor.cc » ('j') | cc/trees/latency_info_swap_promise_monitor.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/surfaces/surface_synchronization_unittest.cc
diff --git a/cc/surfaces/surface_synchronization_unittest.cc b/cc/surfaces/surface_synchronization_unittest.cc
index 9b573261f568a2c9e69a82534b6d811464a8ad07..4dc79c4cd4115853ee948a02293e96a07e26a945 100644
--- a/cc/surfaces/surface_synchronization_unittest.cc
+++ b/cc/surfaces/surface_synchronization_unittest.cc
@@ -749,14 +749,12 @@ TEST_F(SurfaceSynchronizationTest,
const ui::LatencyComponentType latency_type1 =
ui::BROWSER_SNAPSHOT_FRAME_NUMBER_COMPONENT;
const int64_t latency_id1 = 234;
- const int64_t latency_sequence_number1 = 5645432;
const ui::LatencyComponentType latency_type2 = ui::TAB_SHOW_COMPONENT;
const int64_t latency_id2 = 31434351;
- const int64_t latency_sequence_number2 = 663788;
// Submit a frame with latency info
ui::LatencyInfo info;
- info.AddLatencyNumber(latency_type1, latency_id1, latency_sequence_number1);
+ info.AddLatencyNumber(latency_type1, latency_id1);
CompositorFrame frame = MakeCompositorFrame();
frame.metadata.latency_info.push_back(info);
@@ -773,7 +771,7 @@ TEST_F(SurfaceSynchronizationTest,
// Submit another frame with some other latency info and a different
// LocalSurfaceId.
ui::LatencyInfo info2;
- info2.AddLatencyNumber(latency_type2, latency_id2, latency_sequence_number2);
+ info2.AddLatencyNumber(latency_type2, latency_id2);
CompositorFrame frame2 = MakeCompositorFrame();
frame2.metadata.latency_info.push_back(info2);
@@ -803,7 +801,6 @@ TEST_F(SurfaceSynchronizationTest,
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(
@@ -821,14 +818,12 @@ TEST_F(SurfaceSynchronizationTest,
const ui::LatencyComponentType latency_type1 =
ui::BROWSER_SNAPSHOT_FRAME_NUMBER_COMPONENT;
const int64_t latency_id1 = 234;
- const int64_t latency_sequence_number1 = 5645432;
const ui::LatencyComponentType latency_type2 = ui::TAB_SHOW_COMPONENT;
const int64_t latency_id2 = 31434351;
- const int64_t latency_sequence_number2 = 663788;
// Submit a frame with no unresolved dependecy.
ui::LatencyInfo info;
- info.AddLatencyNumber(latency_type1, latency_id1, latency_sequence_number1);
+ info.AddLatencyNumber(latency_type1, latency_id1);
CompositorFrame frame = MakeCompositorFrame();
frame.metadata.latency_info.push_back(info);
@@ -838,7 +833,7 @@ TEST_F(SurfaceSynchronizationTest,
// Submit a frame with unresolved dependencies.
ui::LatencyInfo info2;
- info2.AddLatencyNumber(latency_type2, latency_id2, latency_sequence_number2);
+ info2.AddLatencyNumber(latency_type2, latency_id2);
CompositorFrame frame2 = MakeCompositorFrame({child_id}, empty_surface_ids(),
TransferableResourceArray());
@@ -880,7 +875,6 @@ TEST_F(SurfaceSynchronizationTest,
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(
@@ -898,14 +892,12 @@ TEST_F(SurfaceSynchronizationTest,
const ui::LatencyComponentType latency_type1 =
ui::BROWSER_SNAPSHOT_FRAME_NUMBER_COMPONENT;
const int64_t latency_id1 = 234;
- const int64_t latency_sequence_number1 = 5645432;
const ui::LatencyComponentType latency_type2 = ui::TAB_SHOW_COMPONENT;
const int64_t latency_id2 = 31434351;
- const int64_t latency_sequence_number2 = 663788;
// Submit a frame with no unresolved dependencies.
ui::LatencyInfo info;
- info.AddLatencyNumber(latency_type1, latency_id1, latency_sequence_number1);
+ info.AddLatencyNumber(latency_type1, latency_id1);
CompositorFrame frame = MakeCompositorFrame();
frame.metadata.latency_info.push_back(info);
@@ -922,7 +914,7 @@ TEST_F(SurfaceSynchronizationTest,
// Submit a frame with a new local surface id and with unresolved
// dependencies.
ui::LatencyInfo info2;
- info2.AddLatencyNumber(latency_type2, latency_id2, latency_sequence_number2);
+ info2.AddLatencyNumber(latency_type2, latency_id2);
CompositorFrame frame2 = MakeCompositorFrame({child_id}, empty_surface_ids(),
TransferableResourceArray());
@@ -960,7 +952,6 @@ TEST_F(SurfaceSynchronizationTest,
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(
« no previous file with comments | « cc/surfaces/compositor_frame_sink_support.cc ('k') | cc/trees/latency_info_swap_promise_monitor.cc » ('j') | cc/trees/latency_info_swap_promise_monitor.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698