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

Side by Side Diff: ash/wm/panels/panel_layout_manager_unittest.cc

Issue 2895713002: [mus+ash] Removes WmWindow from ash/wm/mru_window_tracker and overview mode (Closed)
Patch Set: Address nits, unit_tests target compiles 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "ash/wm/panels/panel_layout_manager.h" 5 #include "ash/wm/panels/panel_layout_manager.h"
6 6
7 #include "ash/public/cpp/config.h" 7 #include "ash/public/cpp/config.h"
8 #include "ash/public/cpp/shelf_types.h" 8 #include "ash/public/cpp/shelf_types.h"
9 #include "ash/public/cpp/shell_window_ids.h" 9 #include "ash/public/cpp/shell_window_ids.h"
10 #include "ash/public/cpp/window_properties.h" 10 #include "ash/public/cpp/window_properties.h"
(...skipping 834 matching lines...) Expand 10 before | Expand all | Expand 10 after
845 // w3 is created while in full-screen mode, should only become visible when 845 // w3 is created while in full-screen mode, should only become visible when
846 // we exit fullscreen mode. 846 // we exit fullscreen mode.
847 w3.reset(CreatePanelWindow(bounds)); 847 w3.reset(CreatePanelWindow(bounds));
848 848
849 EXPECT_FALSE(w1->IsVisible()); 849 EXPECT_FALSE(w1->IsVisible());
850 EXPECT_FALSE(w2->IsVisible()); 850 EXPECT_FALSE(w2->IsVisible());
851 EXPECT_FALSE(w3->IsVisible()); 851 EXPECT_FALSE(w3->IsVisible());
852 852
853 // While in full-screen mode, the panel windows should still be in the 853 // While in full-screen mode, the panel windows should still be in the
854 // switchable window list - http://crbug.com/313919. 854 // switchable window list - http://crbug.com/313919.
855 aura::Window::Windows switchable_window_list = WmWindow::ToAuraWindows( 855 aura::Window::Windows switchable_window_list =
856 Shell::Get()->mru_window_tracker()->BuildMruWindowList()); 856 Shell::Get()->mru_window_tracker()->BuildMruWindowList();
857 EXPECT_EQ(3u, switchable_window_list.size()); 857 EXPECT_EQ(3u, switchable_window_list.size());
858 EXPECT_NE(switchable_window_list.end(), 858 EXPECT_NE(switchable_window_list.end(),
859 std::find(switchable_window_list.begin(), 859 std::find(switchable_window_list.begin(),
860 switchable_window_list.end(), w1.get())); 860 switchable_window_list.end(), w1.get()));
861 EXPECT_NE(switchable_window_list.end(), 861 EXPECT_NE(switchable_window_list.end(),
862 std::find(switchable_window_list.begin(), 862 std::find(switchable_window_list.begin(),
863 switchable_window_list.end(), w2.get())); 863 switchable_window_list.end(), w2.get()));
864 EXPECT_NE(switchable_window_list.end(), 864 EXPECT_NE(switchable_window_list.end(),
865 std::find(switchable_window_list.begin(), 865 std::find(switchable_window_list.begin(),
866 switchable_window_list.end(), w3.get())); 866 switchable_window_list.end(), w3.get()));
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
929 touch.set_location(gfx::Point(bounds.x() - 1, bounds.y() + 5)); 929 touch.set_location(gfx::Point(bounds.x() - 1, bounds.y() + 5));
930 target = targeter->FindTargetForEvent(root, &touch); 930 target = targeter->FindTargetForEvent(root, &touch);
931 EXPECT_NE(w.get(), target); 931 EXPECT_NE(w.get(), target);
932 } 932 }
933 933
934 INSTANTIATE_TEST_CASE_P(LtrRtl, 934 INSTANTIATE_TEST_CASE_P(LtrRtl,
935 PanelLayoutManagerTextDirectionTest, 935 PanelLayoutManagerTextDirectionTest,
936 testing::Bool()); 936 testing::Bool());
937 937
938 } // namespace ash 938 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698