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

Side by Side Diff: ash/content/keyboard_overlay/keyboard_overlay_delegate_unittest.cc

Issue 2461053002: Center Keyboard Overlay in the work area (Closed)
Patch Set: Fix tests Created 4 years, 1 month 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
« no previous file with comments | « ash/content/keyboard_overlay/keyboard_overlay_delegate.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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/content/keyboard_overlay/keyboard_overlay_delegate.h" 5 #include "ash/content/keyboard_overlay/keyboard_overlay_delegate.h"
6 6
7 #include "ash/common/shelf/wm_shelf.h" 7 #include "ash/common/shelf/wm_shelf.h"
8 #include "ash/public/cpp/shelf_types.h" 8 #include "ash/public/cpp/shelf_types.h"
9 #include "ash/shell.h" 9 #include "ash/shell.h"
10 #include "ash/test/ash_test_base.h" 10 #include "ash/test/ash_test_base.h"
(...skipping 22 matching lines...) Expand all
33 // Verifies we can show and close the widget for the overlay dialog. 33 // Verifies we can show and close the widget for the overlay dialog.
34 TEST_P(KeyboardOverlayDelegateTest, ShowAndClose) { 34 TEST_P(KeyboardOverlayDelegateTest, ShowAndClose) {
35 if (!SupportsMultipleDisplays()) 35 if (!SupportsMultipleDisplays())
36 return; 36 return;
37 37
38 UpdateDisplay("500x400,300x200"); 38 UpdateDisplay("500x400,300x200");
39 GetPrimaryShelf()->SetAlignment(shelf_alignment()); 39 GetPrimaryShelf()->SetAlignment(shelf_alignment());
40 KeyboardOverlayDelegate delegate(base::ASCIIToUTF16("Title"), 40 KeyboardOverlayDelegate delegate(base::ASCIIToUTF16("Title"),
41 GURL("chrome://keyboardoverlay/")); 41 GURL("chrome://keyboardoverlay/"));
42 // Showing the dialog creates a widget. 42 // Showing the dialog creates a widget.
43 views::Widget* widget = delegate.Show(NULL); 43 views::Widget* widget = delegate.Show(nullptr);
44 EXPECT_TRUE(widget); 44 EXPECT_TRUE(widget);
45 45
46 // The widget is on the primary root window. 46 // The widget is on the primary root window.
47 EXPECT_EQ(Shell::GetPrimaryRootWindow(), 47 EXPECT_EQ(Shell::GetPrimaryRootWindow(),
48 widget->GetNativeWindow()->GetRootWindow()); 48 widget->GetNativeWindow()->GetRootWindow());
49 49
50 // The widget is horizontally centered at the bottom of the work area. 50 // The widget is horizontally and vertically centered in the work area.
51 gfx::Rect work_area = 51 gfx::Rect work_area =
52 display::Screen::GetScreen()->GetPrimaryDisplay().work_area(); 52 display::Screen::GetScreen()->GetPrimaryDisplay().work_area();
53 gfx::Rect bounds = widget->GetRestoredBounds(); 53 gfx::Rect bounds = widget->GetRestoredBounds();
54 EXPECT_EQ(work_area.CenterPoint().x(), bounds.CenterPoint().x()); 54 EXPECT_EQ(work_area.CenterPoint().x(), bounds.CenterPoint().x());
55 EXPECT_EQ(work_area.bottom(), bounds.bottom()); 55 EXPECT_EQ(work_area.y() + (work_area.height() - bounds.height()) / 2,
56 bounds.y());
56 57
57 // Clean up. 58 // Clean up.
58 widget->CloseNow(); 59 widget->CloseNow();
59 } 60 }
60 61
61 // Tests run three times - for all possible values of shelf alignment 62 // Tests run four times - for all possible values of shelf alignment
62 INSTANTIATE_TEST_CASE_P(ShelfAlignmentAny, 63 INSTANTIATE_TEST_CASE_P(ShelfAlignmentAny,
63 KeyboardOverlayDelegateTest, 64 KeyboardOverlayDelegateTest,
64 testing::Values(SHELF_ALIGNMENT_BOTTOM, 65 testing::Values(SHELF_ALIGNMENT_BOTTOM,
65 SHELF_ALIGNMENT_LEFT, 66 SHELF_ALIGNMENT_LEFT,
66 SHELF_ALIGNMENT_RIGHT, 67 SHELF_ALIGNMENT_RIGHT,
67 SHELF_ALIGNMENT_BOTTOM_LOCKED)); 68 SHELF_ALIGNMENT_BOTTOM_LOCKED));
68 69
69 } // namespace ash 70 } // namespace ash
OLDNEW
« no previous file with comments | « ash/content/keyboard_overlay/keyboard_overlay_delegate.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698