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

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

Issue 2497303002: Fix ws::Display initialization order. (Closed)
Patch Set: Fix use after move in test. Created 4 years, 1 month 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.cc ('k') | services/ui/ws/window_tree_host_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ws/user_display_manager_unittest.cc
diff --git a/services/ui/ws/user_display_manager_unittest.cc b/services/ui/ws/user_display_manager_unittest.cc
index ee827f0cf209e2818469b46ff184d4df4c36ef53..1380e9f9bdee720e216bc06dcb63a41d99fccb8f 100644
--- a/services/ui/ws/user_display_manager_unittest.cc
+++ b/services/ui/ws/user_display_manager_unittest.cc
@@ -200,8 +200,8 @@ TEST_F(UserDisplayManagerTest, AddRemoveDisplay) {
display_manager_observer1.GetAndClearObserverCalls());
// Add another display.
- Display* display2 = new Display(window_server(), PlatformDisplayInitParams());
- display2->Init(nullptr);
+ Display* display2 = new Display(window_server());
+ display2->Init(PlatformDisplayInitParams(), nullptr);
// Observer should be notified immediately as frame decorations were set.
EXPECT_EQ("OnDisplaysChanged 2",
« no previous file with comments | « services/ui/ws/test_utils.cc ('k') | services/ui/ws/window_tree_host_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698