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

Unified Diff: services/ui/ws/test_utils.cc

Issue 2886873002: Only send the FrameSinkId to client when it is necessary (Closed)
Patch Set: Address review issues. Created 3 years, 7 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 | « services/ui/ws/test_utils.h ('k') | services/ui/ws/window_manager_client_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ws/test_utils.cc
diff --git a/services/ui/ws/test_utils.cc b/services/ui/ws/test_utils.cc
index 275226a2d15929bf7adc592cdb5c37dfab7ad407..f9735fe32b1b7f9c1befa4ccab110e3945a27829 100644
--- a/services/ui/ws/test_utils.cc
+++ b/services/ui/ws/test_utils.cc
@@ -226,7 +226,6 @@ void TestWindowManager::WmNewDisplayAdded(
const display::Display& display,
ui::mojom::WindowDataPtr root,
bool drawn,
- const cc::FrameSinkId& frame_sink_id,
const base::Optional<cc::LocalSurfaceId>& local_surface_id) {
display_added_count_++;
}
@@ -306,10 +305,9 @@ void TestWindowTreeClient::OnEmbed(
int64_t display_id,
Id focused_window_id,
bool drawn,
- const cc::FrameSinkId& frame_sink_id,
const base::Optional<cc::LocalSurfaceId>& local_surface_id) {
// TODO(sky): add test coverage of |focused_window_id|.
- tracker_.OnEmbed(client_id, std::move(root), drawn, frame_sink_id);
+ tracker_.OnEmbed(client_id, std::move(root), drawn);
}
void TestWindowTreeClient::OnEmbeddedAppDisconnected(uint32_t window) {
@@ -336,9 +334,8 @@ void TestWindowTreeClient::OnTopLevelCreated(
mojom::WindowDataPtr data,
int64_t display_id,
bool drawn,
- const cc::FrameSinkId& frame_sink_id,
const base::Optional<cc::LocalSurfaceId>& local_surface_id) {
- tracker_.OnTopLevelCreated(change_id, std::move(data), drawn, frame_sink_id);
+ tracker_.OnTopLevelCreated(change_id, std::move(data), drawn);
}
void TestWindowTreeClient::OnWindowBoundsChanged(
« no previous file with comments | « services/ui/ws/test_utils.h ('k') | services/ui/ws/window_manager_client_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698