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

Unified Diff: ash/wm/splitview/split_view_controller_unittest.cc

Issue 2978273002: Revert of Cros Tablet Window management - Split Screen part II (Closed)
Patch Set: Created 3 years, 5 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 | « ash/wm/splitview/split_view_controller.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/wm/splitview/split_view_controller_unittest.cc
diff --git a/ash/wm/splitview/split_view_controller_unittest.cc b/ash/wm/splitview/split_view_controller_unittest.cc
index a726183ce424ccf91bc500781c8994d5e3185632..2f1a196ef127b07a8bd7a6c01b0b0538b0ba5f18 100644
--- a/ash/wm/splitview/split_view_controller_unittest.cc
+++ b/ash/wm/splitview/split_view_controller_unittest.cc
@@ -6,7 +6,6 @@
#include "ash/shell.h"
#include "ash/test/ash_test_base.h"
-#include "ash/wm/overview/window_selector_controller.h"
#include "ash/wm/window_state.h"
#include "ash/wm/window_util.h"
#include "ash/wm/wm_event.h"
@@ -26,16 +25,6 @@
}
void EndSplitView() { split_view_controller()->EndSplitView(); }
-
- void ToggleOverview() {
- Shell::Get()->window_selector_controller()->ToggleOverview();
- }
-
- std::vector<aura::Window*> GetWindowsInOverviewGrids() {
- return Shell::Get()
- ->window_selector_controller()
- ->GetWindowsListInOverviewGridsForTesting();
- }
SplitViewController* split_view_controller() {
return Shell::Get()->split_view_controller();
@@ -64,7 +53,7 @@
EXPECT_EQ(split_view_controller()->IsSplitViewModeActive(), true);
EXPECT_EQ(window1->GetBoundsInScreen(),
split_view_controller()->GetSnappedWindowBoundsInScreen(
- window1.get(), SplitViewController::LEFT));
+ window1.get(), SplitViewController::LEFT_SNAPPED));
split_view_controller()->SnapWindow(window2.get(),
SplitViewController::RIGHT);
@@ -75,7 +64,7 @@
EXPECT_EQ(split_view_controller()->IsSplitViewModeActive(), true);
EXPECT_EQ(window2->GetBoundsInScreen(),
split_view_controller()->GetSnappedWindowBoundsInScreen(
- window2.get(), SplitViewController::RIGHT));
+ window2.get(), SplitViewController::RIGHT_SNAPPED));
EndSplitView();
EXPECT_EQ(split_view_controller()->state(), SplitViewController::NO_SNAP);
@@ -171,53 +160,4 @@
SplitViewController::BOTH_SNAPPED);
}
-// Tests that if split view mode and overview mode are active at the same time,
-// i.e., half of the screen is occupied by a snapped window and half of the
-// screen is occupied by the overview windows grid, the next activatable window
-// will be picked to snap when exiting the overview mode.
-TEST_F(SplitViewControllerTest, ExitOverviewTest) {
- const gfx::Rect bounds(0, 0, 400, 400);
- std::unique_ptr<aura::Window> window1(CreateWindow(bounds));
- std::unique_ptr<aura::Window> window2(CreateWindow(bounds));
- std::unique_ptr<aura::Window> window3(CreateWindow(bounds));
- EXPECT_EQ(split_view_controller()->IsSplitViewModeActive(), false);
-
- ToggleOverview();
- split_view_controller()->SnapWindow(window1.get(), SplitViewController::LEFT);
- EXPECT_EQ(split_view_controller()->IsSplitViewModeActive(), true);
- EXPECT_EQ(split_view_controller()->state(),
- SplitViewController::LEFT_SNAPPED);
- EXPECT_EQ(split_view_controller()->left_window(), window1.get());
-
- ToggleOverview();
- EXPECT_EQ(split_view_controller()->state(),
- SplitViewController::BOTH_SNAPPED);
- EXPECT_EQ(split_view_controller()->right_window(), window3.get());
-}
-
-// Tests that if split view mode is active when entering overview, the overview
-// windows grid should show in the non-default side of the screen, and the
-// default snapped window should not be shown in the overview window grid.
-TEST_F(SplitViewControllerTest, EnterOverviewTest) {
- const gfx::Rect bounds(0, 0, 400, 400);
- std::unique_ptr<aura::Window> window1(CreateWindow(bounds));
- std::unique_ptr<aura::Window> window2(CreateWindow(bounds));
- std::unique_ptr<aura::Window> window3(CreateWindow(bounds));
-
- split_view_controller()->SnapWindow(window1.get(), SplitViewController::LEFT);
- split_view_controller()->SnapWindow(window2.get(),
- SplitViewController::RIGHT);
- EXPECT_EQ(split_view_controller()->state(),
- SplitViewController::BOTH_SNAPPED);
- EXPECT_EQ(split_view_controller()->GetDefaultSnappedWindow(), window1.get());
-
- ToggleOverview();
- EXPECT_EQ(split_view_controller()->state(),
- SplitViewController::LEFT_SNAPPED);
- std::vector<aura::Window*> windows = GetWindowsInOverviewGrids();
- auto iter = std::find(windows.begin(), windows.end(),
- split_view_controller()->GetDefaultSnappedWindow());
- EXPECT_TRUE(iter == windows.end());
-}
-
} // namespace ash
« no previous file with comments | « ash/wm/splitview/split_view_controller.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698