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

Unified Diff: ash/shelf/shelf_widget_unittest.cc

Issue 2692663002: Gut ash::MaterialDesignController, and remove the about:flags entry. (Closed)
Patch Set: yet another rebase Created 3 years, 10 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/screen_util_unittest.cc ('k') | ash/shell/content/client/shell_browser_main_parts.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/shelf/shelf_widget_unittest.cc
diff --git a/ash/shelf/shelf_widget_unittest.cc b/ash/shelf/shelf_widget_unittest.cc
index 42c07ac666b40cebae102bb60e2b5f52cda83400..69e43154515759ee46e5f8125d6427f56cdc8e5e 100644
--- a/ash/shelf/shelf_widget_unittest.cc
+++ b/ash/shelf/shelf_widget_unittest.cc
@@ -4,7 +4,6 @@
#include "ash/common/shelf/shelf_widget.h"
-#include "ash/common/material_design/material_design_controller.h"
#include "ash/common/shelf/shelf_constants.h"
#include "ash/common/shelf/shelf_layout_manager.h"
#include "ash/common/shelf/shelf_view.h"
@@ -14,7 +13,7 @@
#include "ash/common/wm_window.h"
#include "ash/root_window_controller.h"
#include "ash/shell.h"
-#include "ash/test/ash_md_test_base.h"
+#include "ash/test/ash_test_base.h"
#include "ash/test/ash_test_helper.h"
#include "ash/test/shelf_view_test_api.h"
#include "ash/wm/window_util.h"
@@ -38,14 +37,7 @@ ShelfLayoutManager* GetShelfLayoutManager() {
} // namespace
-using ShelfWidgetTest = test::AshMDTestBase;
-
-INSTANTIATE_TEST_CASE_P(
- /* prefix intentionally left blank due to only one parameterization */,
- ShelfWidgetTest,
- testing::Values(MaterialDesignController::NON_MATERIAL,
- MaterialDesignController::MATERIAL_NORMAL,
- MaterialDesignController::MATERIAL_EXPERIMENTAL));
+using ShelfWidgetTest = test::AshTestBase;
void TestLauncherAlignment(WmWindow* root,
ShelfAlignment alignment,
@@ -55,89 +47,81 @@ void TestLauncherAlignment(WmWindow* root,
root->GetDisplayNearestWindow().work_area().ToString());
}
-TEST_P(ShelfWidgetTest, TestAlignment) {
- // Note that for a left- and right-aligned shelf, this offset must be
- // applied to a maximized window's width rather than its height.
- const int offset = GetMdMaximizedWindowHeightOffset();
+TEST_F(ShelfWidgetTest, TestAlignment) {
const int kShelfSize = GetShelfConstant(SHELF_SIZE);
UpdateDisplay("400x400");
{
SCOPED_TRACE("Single Bottom");
TestLauncherAlignment(WmShell::Get()->GetPrimaryRootWindow(),
- SHELF_ALIGNMENT_BOTTOM,
- gfx::Rect(0, 0, 400, 353 + offset));
+ SHELF_ALIGNMENT_BOTTOM, gfx::Rect(0, 0, 400, 352));
}
{
SCOPED_TRACE("Single Locked");
TestLauncherAlignment(WmShell::Get()->GetPrimaryRootWindow(),
SHELF_ALIGNMENT_BOTTOM_LOCKED,
- gfx::Rect(0, 0, 400, 353 + offset));
+ gfx::Rect(0, 0, 400, 352));
}
{
SCOPED_TRACE("Single Right");
TestLauncherAlignment(WmShell::Get()->GetPrimaryRootWindow(),
- SHELF_ALIGNMENT_RIGHT,
- gfx::Rect(0, 0, 353 + offset, 400));
+ SHELF_ALIGNMENT_RIGHT, gfx::Rect(0, 0, 352, 400));
}
{
SCOPED_TRACE("Single Left");
TestLauncherAlignment(WmShell::Get()->GetPrimaryRootWindow(),
SHELF_ALIGNMENT_LEFT,
- gfx::Rect(kShelfSize, 0, 353 + offset, 400));
+ gfx::Rect(kShelfSize, 0, 352, 400));
}
}
-TEST_P(ShelfWidgetTest, TestAlignmentForMultipleDisplays) {
- // Note that for a left- and right-aligned shelf, this offset must be
- // applied to a maximized window's width rather than its height.
- const int offset = GetMdMaximizedWindowHeightOffset();
+TEST_F(ShelfWidgetTest, TestAlignmentForMultipleDisplays) {
const int kShelfSize = GetShelfConstant(SHELF_SIZE);
UpdateDisplay("300x300,500x500");
std::vector<WmWindow*> root_windows = WmShell::Get()->GetAllRootWindows();
{
SCOPED_TRACE("Primary Bottom");
TestLauncherAlignment(root_windows[0], SHELF_ALIGNMENT_BOTTOM,
- gfx::Rect(0, 0, 300, 253 + offset));
+ gfx::Rect(0, 0, 300, 252));
}
{
SCOPED_TRACE("Primary Locked");
TestLauncherAlignment(root_windows[0], SHELF_ALIGNMENT_BOTTOM_LOCKED,
- gfx::Rect(0, 0, 300, 253 + offset));
+ gfx::Rect(0, 0, 300, 252));
}
{
SCOPED_TRACE("Primary Right");
TestLauncherAlignment(root_windows[0], SHELF_ALIGNMENT_RIGHT,
- gfx::Rect(0, 0, 253 + offset, 300));
+ gfx::Rect(0, 0, 252, 300));
}
{
SCOPED_TRACE("Primary Left");
TestLauncherAlignment(root_windows[0], SHELF_ALIGNMENT_LEFT,
- gfx::Rect(kShelfSize, 0, 253 + offset, 300));
+ gfx::Rect(kShelfSize, 0, 252, 300));
}
{
SCOPED_TRACE("Secondary Bottom");
TestLauncherAlignment(root_windows[1], SHELF_ALIGNMENT_BOTTOM,
- gfx::Rect(300, 0, 500, 453 + offset));
+ gfx::Rect(300, 0, 500, 452));
}
{
SCOPED_TRACE("Secondary Locked");
TestLauncherAlignment(root_windows[1], SHELF_ALIGNMENT_BOTTOM_LOCKED,
- gfx::Rect(300, 0, 500, 453 + offset));
+ gfx::Rect(300, 0, 500, 452));
}
{
SCOPED_TRACE("Secondary Right");
TestLauncherAlignment(root_windows[1], SHELF_ALIGNMENT_RIGHT,
- gfx::Rect(300, 0, 453 + offset, 500));
+ gfx::Rect(300, 0, 452, 500));
}
{
SCOPED_TRACE("Secondary Left");
TestLauncherAlignment(root_windows[1], SHELF_ALIGNMENT_LEFT,
- gfx::Rect(300 + kShelfSize, 0, 453 + offset, 500));
+ gfx::Rect(300 + kShelfSize, 0, 452, 500));
}
}
// Makes sure the shelf is initially sized correctly.
-TEST_P(ShelfWidgetTest, LauncherInitiallySized) {
+TEST_F(ShelfWidgetTest, LauncherInitiallySized) {
ShelfWidget* shelf_widget = GetShelfWidget();
ShelfLayoutManager* shelf_layout_manager = GetShelfLayoutManager();
ASSERT_TRUE(shelf_layout_manager);
@@ -152,7 +136,7 @@ TEST_P(ShelfWidgetTest, LauncherInitiallySized) {
}
// Verifies when the shell is deleted with a full screen window we don't crash.
-TEST_P(ShelfWidgetTest, DontReferenceShelfAfterDeletion) {
+TEST_F(ShelfWidgetTest, DontReferenceShelfAfterDeletion) {
views::Widget* widget = new views::Widget;
views::Widget::InitParams params(views::Widget::InitParams::TYPE_WINDOW);
params.bounds = gfx::Rect(0, 0, 200, 200);
@@ -165,7 +149,7 @@ TEST_P(ShelfWidgetTest, DontReferenceShelfAfterDeletion) {
// Verifies shelf is created with correct size after user login and when its
// container and status widget has finished sizing.
// See http://crbug.com/252533
-TEST_P(ShelfWidgetTest, ShelfInitiallySizedAfterLogin) {
+TEST_F(ShelfWidgetTest, ShelfInitiallySizedAfterLogin) {
SetUserLoggedIn(false);
UpdateDisplay("300x200,400x300");
@@ -202,7 +186,7 @@ TEST_P(ShelfWidgetTest, ShelfInitiallySizedAfterLogin) {
// Tests that the shelf lets mouse-events close to the edge fall through to the
// window underneath.
-TEST_P(ShelfWidgetTest, ShelfEdgeOverlappingWindowHitTestMouse) {
+TEST_F(ShelfWidgetTest, ShelfEdgeOverlappingWindowHitTestMouse) {
UpdateDisplay("400x400");
ShelfWidget* shelf_widget = GetShelfWidget();
gfx::Rect shelf_bounds = shelf_widget->GetWindowBoundsInScreen();
@@ -287,7 +271,7 @@ TEST_P(ShelfWidgetTest, ShelfEdgeOverlappingWindowHitTestMouse) {
// Tests that the shelf has a slightly larger hit-region for touch-events when
// it's in the auto-hidden state.
-TEST_P(ShelfWidgetTest, HiddenShelfHitTestTouch) {
+TEST_F(ShelfWidgetTest, HiddenShelfHitTestTouch) {
WmShelf* shelf = GetPrimaryShelf();
ShelfWidget* shelf_widget = GetShelfWidget();
gfx::Rect shelf_bounds = shelf_widget->GetWindowBoundsInScreen();
@@ -400,35 +384,28 @@ class ShelfWidgetTestWithInitializer : public ShelfWidgetTest {
} // namespace
-INSTANTIATE_TEST_CASE_P(
- /* prefix intentionally left blank due to only one parameterization */,
- ShelfWidgetTestWithInitializer,
- testing::Values(MaterialDesignController::NON_MATERIAL,
- MaterialDesignController::MATERIAL_NORMAL,
- MaterialDesignController::MATERIAL_EXPERIMENTAL));
-
-TEST_P(ShelfWidgetTestWithInitializer, CreateAutoHideAlwaysShelf) {
+TEST_F(ShelfWidgetTestWithInitializer, CreateAutoHideAlwaysShelf) {
// The actual auto hide state is shown because there are no open windows.
TestCreateShelfWithInitialValues(SHELF_ALIGNMENT_BOTTOM,
SHELF_AUTO_HIDE_BEHAVIOR_ALWAYS,
SHELF_AUTO_HIDE, SHELF_AUTO_HIDE_SHOWN);
}
-TEST_P(ShelfWidgetTestWithInitializer, CreateAutoHideNeverShelf) {
+TEST_F(ShelfWidgetTestWithInitializer, CreateAutoHideNeverShelf) {
// The auto hide state 'HIDDEN' is returned for any non-auto-hide behavior.
TestCreateShelfWithInitialValues(SHELF_ALIGNMENT_LEFT,
SHELF_AUTO_HIDE_BEHAVIOR_NEVER,
SHELF_VISIBLE, SHELF_AUTO_HIDE_HIDDEN);
}
-TEST_P(ShelfWidgetTestWithInitializer, CreateAutoHideAlwaysHideShelf) {
+TEST_F(ShelfWidgetTestWithInitializer, CreateAutoHideAlwaysHideShelf) {
// The auto hide state 'HIDDEN' is returned for any non-auto-hide behavior.
TestCreateShelfWithInitialValues(SHELF_ALIGNMENT_RIGHT,
SHELF_AUTO_HIDE_ALWAYS_HIDDEN, SHELF_HIDDEN,
SHELF_AUTO_HIDE_HIDDEN);
}
-TEST_P(ShelfWidgetTestWithInitializer, CreateLockedShelf) {
+TEST_F(ShelfWidgetTestWithInitializer, CreateLockedShelf) {
// The auto hide state 'HIDDEN' is returned for any non-auto-hide behavior.
TestCreateShelfWithInitialValues(SHELF_ALIGNMENT_BOTTOM_LOCKED,
SHELF_AUTO_HIDE_BEHAVIOR_NEVER,
« no previous file with comments | « ash/screen_util_unittest.cc ('k') | ash/shell/content/client/shell_browser_main_parts.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698