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

Side by Side Diff: components/mus/ws/window_manager_client_shelltest.cc

Issue 1771393002: Convert mus apptests to shelltests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 827 matching lines...) Expand 10 before | Expand all | Expand 10 after
838 EXPECT_EQ(child21->id(), window_manager()->GetFocusedWindow()->id()); 838 EXPECT_EQ(child21->id(), window_manager()->GetFocusedWindow()->id());
839 EXPECT_EQ(child21->id(), embedded2->GetFocusedWindow()->id()); 839 EXPECT_EQ(child21->id(), embedded2->GetFocusedWindow()->id());
840 EXPECT_TRUE(WaitForNoWindowToHaveFocus(embedded1)); 840 EXPECT_TRUE(WaitForNoWindowToHaveFocus(embedded1));
841 EXPECT_EQ(nullptr, embedded1->GetFocusedWindow()); 841 EXPECT_EQ(nullptr, embedded1->GetFocusedWindow());
842 EXPECT_GT(ValidIndexOf(parent->children(), child2), 842 EXPECT_GT(ValidIndexOf(parent->children(), child2),
843 ValidIndexOf(parent->children(), child1)); 843 ValidIndexOf(parent->children(), child1));
844 EXPECT_GT(ValidIndexOf(parent->children(), child3), 844 EXPECT_GT(ValidIndexOf(parent->children(), child3),
845 ValidIndexOf(parent->children(), child1)); 845 ValidIndexOf(parent->children(), child1));
846 } 846 }
847 847
848 // Very flaky: http://crbug.com/592313. 848 TEST_F(WindowServerTest, ActivationNext) {
849 TEST_F(WindowServerTest, DISABLED_ActivationNext) {
850 Window* parent = GetFirstWMRoot(); 849 Window* parent = GetFirstWMRoot();
851 Window* child1 = NewVisibleWindow(parent, window_manager()); 850 Window* child1 = NewVisibleWindow(parent, window_manager());
852 Window* child2 = NewVisibleWindow(parent, window_manager()); 851 Window* child2 = NewVisibleWindow(parent, window_manager());
853 Window* child3 = NewVisibleWindow(parent, window_manager()); 852 Window* child3 = NewVisibleWindow(parent, window_manager());
854 853
855 WindowTreeConnection* embedded1 = Embed(child1).connection; 854 WindowTreeConnection* embedded1 = Embed(child1).connection;
856 ASSERT_NE(nullptr, embedded1); 855 ASSERT_NE(nullptr, embedded1);
857 WindowTreeConnection* embedded2 = Embed(child2).connection; 856 WindowTreeConnection* embedded2 = Embed(child2).connection;
858 ASSERT_NE(nullptr, embedded2); 857 ASSERT_NE(nullptr, embedded2);
859 WindowTreeConnection* embedded3 = Embed(child3).connection; 858 WindowTreeConnection* embedded3 = Embed(child3).connection;
(...skipping 313 matching lines...) Expand 10 before | Expand all | Expand 10 after
1173 ASSERT_TRUE(window_in_wm); 1172 ASSERT_TRUE(window_in_wm);
1174 1173
1175 // Change the bounds in the wm, and make sure the child sees it. 1174 // Change the bounds in the wm, and make sure the child sees it.
1176 window_in_wm->SetBounds(gfx::Rect(1, 11, 12, 101)); 1175 window_in_wm->SetBounds(gfx::Rect(1, 11, 12, 101));
1177 ASSERT_TRUE(WaitForBoundsToChange(window_in_second_connection)); 1176 ASSERT_TRUE(WaitForBoundsToChange(window_in_second_connection));
1178 EXPECT_EQ(gfx::Rect(1, 11, 12, 101), window_in_second_connection->bounds()); 1177 EXPECT_EQ(gfx::Rect(1, 11, 12, 101), window_in_second_connection->bounds());
1179 } 1178 }
1180 1179
1181 } // namespace ws 1180 } // namespace ws
1182 } // namespace mus 1181 } // namespace mus
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698