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

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

Issue 2582823002: WIP: Surface Synchronization System
Patch Set: Only create ClientSurfaceEmbedder if window is visible. Trash it otherwise. Created 3 years, 11 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: services/ui/ws/window_manager_state_unittest.cc
diff --git a/services/ui/ws/window_manager_state_unittest.cc b/services/ui/ws/window_manager_state_unittest.cc
index 6a03c5d90a2791b7625aeb8496f81616bb12c652..cfded7d30ab4a3cb775079fce47327f207759e45 100644
--- a/services/ui/ws/window_manager_state_unittest.cc
+++ b/services/ui/ws/window_manager_state_unittest.cc
@@ -217,7 +217,8 @@ TEST_F(WindowManagerStateTest, PostTargetAccelerator) {
TEST_F(WindowManagerStateTest, PreTargetConsumed) {
// Set up two trees with focus on a child in the second.
const ClientWindowId child_window_id(11);
- window_tree()->NewWindow(child_window_id, ServerWindow::Properties());
+ window_tree()->NewWindow(child_window_id, ServerWindow::Properties(),
+ cc::LocalFrameId());
ServerWindow* child_window =
window_tree()->GetWindowByClientId(child_window_id);
window_tree()->AddWindow(FirstRootId(window_tree()), child_window_id);
@@ -426,7 +427,8 @@ TEST_F(WindowManagerStateTest, InterceptingEmbedderReceivesEvents) {
ServerWindow* embedder_root = window();
const ClientWindowId embed_window_id(
WindowIdToTransportId(WindowId(embedder_tree->id(), 12)));
- embedder_tree->NewWindow(embed_window_id, ServerWindow::Properties());
+ embedder_tree->NewWindow(embed_window_id, ServerWindow::Properties(),
+ cc::LocalFrameId());
ServerWindow* embedder_window =
embedder_tree->GetWindowByClientId(embed_window_id);
ASSERT_TRUE(embedder_tree->AddWindow(
@@ -482,7 +484,8 @@ TEST_F(WindowManagerStateTest, InterceptingEmbedderReceivesEvents) {
// Embed another tree in the embedded tree.
const ClientWindowId nested_embed_window_id(
WindowIdToTransportId(WindowId(embed_tree->id(), 23)));
- embed_tree->NewWindow(nested_embed_window_id, ServerWindow::Properties());
+ embed_tree->NewWindow(nested_embed_window_id, ServerWindow::Properties(),
+ cc::LocalFrameId());
const ClientWindowId embed_root_id(
WindowIdToTransportId((*embed_tree->roots().begin())->id()));
ASSERT_TRUE(embed_tree->AddWindow(embed_root_id, nested_embed_window_id));

Powered by Google App Engine
This is Rietveld 408576698