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

Unified Diff: ash/wm/overview/window_selector_unittest.cc

Issue 2808723004: Renames WmShell to ShellPort (Closed)
Patch Set: feedback Created 3 years, 8 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: ash/wm/overview/window_selector_unittest.cc
diff --git a/ash/wm/overview/window_selector_unittest.cc b/ash/wm/overview/window_selector_unittest.cc
index 85c07bcf1b2dd7a358579f0a5d9f67bd91cad27e..5e37cac16de1a49905c2c9114bcfc29b1624b614 100644
--- a/ash/wm/overview/window_selector_unittest.cc
+++ b/ash/wm/overview/window_selector_unittest.cc
@@ -10,6 +10,7 @@
#include "ash/drag_drop/drag_drop_controller.h"
#include "ash/shelf/wm_shelf.h"
#include "ash/shell.h"
+#include "ash/shell_port.h"
#include "ash/system/tray/system_tray.h"
#include "ash/test/ash_test_base.h"
#include "ash/test/shelf_view_test_api.h"
@@ -27,7 +28,6 @@
#include "ash/wm/window_util.h"
#include "ash/wm/wm_event.h"
#include "ash/wm/workspace/workspace_window_resizer.h"
-#include "ash/wm_shell.h"
#include "ash/wm_window.h"
#include "base/strings/utf_string_conversions.h"
#include "base/test/user_action_tester.h"
@@ -341,7 +341,7 @@ class WindowSelectorTest : public test::AshTestBase {
TEST_F(WindowSelectorTest, OverviewScreenRotation) {
// TODO: fails in mash because rotation is not supported.
// http://crbug.com/695556
- if (WmShell::Get()->IsRunningInMash())
+ if (ShellPort::Get()->IsRunningInMash())
return;
gfx::Rect bounds(0, 0, 400, 300);
@@ -419,7 +419,7 @@ TEST_F(WindowSelectorTest, Basic) {
EXPECT_TRUE(wm::IsActiveWindow(window2.get()));
EXPECT_EQ(window2.get(), wm::GetFocusedWindow());
// TODO: mash doesn't support CursorClient. http://crbug.com/637853.
- if (!WmShell::Get()->IsRunningInMash()) {
+ if (!ShellPort::Get()->IsRunningInMash()) {
// Hide the cursor before entering overview to test that it will be shown.
aura::client::GetCursorClient(root_window)->HideCursor();
}
@@ -439,7 +439,7 @@ TEST_F(WindowSelectorTest, Basic) {
EXPECT_EQ(window1.get(), wm::GetFocusedWindow());
// TODO: mash doesn't support CursorClient. http://crbug.com/637853.
- if (!WmShell::Get()->IsRunningInMash()) {
+ if (!ShellPort::Get()->IsRunningInMash()) {
// Cursor should have been unlocked.
EXPECT_FALSE(aura::client::GetCursorClient(root_window)->IsCursorLocked());
}
@@ -555,7 +555,7 @@ TEST_F(WindowSelectorTest, BasicGesture) {
// in overview mode which is different from the previously-active window.
TEST_F(WindowSelectorTest, ActiveWindowChangedUserActionRecorded) {
// TODO: fails because of metrics. http://crbug.com/698129.
- if (WmShell::Get()->IsRunningInMash())
+ if (ShellPort::Get()->IsRunningInMash())
return;
base::UserActionTester user_action_tester;
@@ -930,7 +930,7 @@ TEST_F(WindowSelectorTest, FullscreenWindowMaximizeMode) {
// Tests that beginning window selection hides the app list.
TEST_F(WindowSelectorTest, SelectingHidesAppList) {
// TODO: fails in mash because of AppListPresenter. http://crbug.com/696028.
- if (WmShell::Get()->IsRunningInMash())
+ if (ShellPort::Get()->IsRunningInMash())
return;
gfx::Rect bounds(0, 0, 400, 400);
@@ -1223,7 +1223,7 @@ TEST_F(WindowSelectorTest, Shutdown) {
TEST_F(WindowSelectorTest, RemoveDisplay) {
// TODO: hits CHECK in stl as order of |ShelfModel::items_| is wrong.
// http://crbug.com/698878.
- if (WmShell::Get()->IsRunningInMash())
+ if (ShellPort::Get()->IsRunningInMash())
return;
UpdateDisplay("400x400,400x400");
@@ -1309,7 +1309,7 @@ TEST_F(WindowSelectorTest, CreateLabelUnderWindow) {
TEST_F(WindowSelectorTest, DisplayOrientationChanged) {
// TODO: fails in mash because rotation is not supported.
// http://crbug.com/695556
- if (WmShell::Get()->IsRunningInMash())
+ if (ShellPort::Get()->IsRunningInMash())
return;
aura::Window* root_window = Shell::Get()->GetPrimaryRootWindow();
@@ -1444,7 +1444,7 @@ TEST_F(WindowSelectorTest, BasicMultiMonitorArrowKeyNavigation) {
TEST_F(WindowSelectorTest, MultiMonitorReversedOrder) {
// TODO: SetLayoutForCurrentDisplays() needs to ported to mash.
// http://crbug.com/698043.
- if (WmShell::Get()->IsRunningInMash())
+ if (ShellPort::Get()->IsRunningInMash())
return;
UpdateDisplay("400x400,400x400");

Powered by Google App Engine
This is Rietveld 408576698