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

Unified Diff: ui/wm/core/ime_util_chromeos_unittest.cc

Issue 2553603002: New accessibility virtual keyboard behavior in non-sticky mode. (Closed)
Patch Set: Renamed to ime_util_chromeos and added the unittests 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
« ui/wm/core/ime_util_chromeos.cc ('K') | « ui/wm/core/ime_util_chromeos.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/wm/core/ime_util_chromeos_unittest.cc
diff --git a/ui/wm/core/ime_util_chromeos_unittest.cc b/ui/wm/core/ime_util_chromeos_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e694e92b367ef65e5f9be8890fd377fec162ffe9
--- /dev/null
+++ b/ui/wm/core/ime_util_chromeos_unittest.cc
@@ -0,0 +1,138 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "ui/wm/core/ime_util_chromeos.h"
+
+#include "base/command_line.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "ui/aura/test/aura_test_base.h"
+#include "ui/aura/test/test_windows.h"
+#include "ui/base/ui_base_switches.h"
+#include "ui/wm/core/default_screen_position_client.h"
+
+namespace wm {
+
+class ImeUtilChromeosTest : public aura::test::AuraTestBase {
+ public:
+ void SetUp() override {
+ AuraTestBase::SetUp();
+ screen_position_client_ = base::MakeUnique<DefaultScreenPositionClient>();
+ aura::client::SetScreenPositionClient(root_window(),
+ screen_position_client_.get());
+
+ base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
+ if (!command_line->HasSwitch(::switches::kUseNewVirtualKeyboardBehavior))
+ command_line->AppendSwitch(::switches::kUseNewVirtualKeyboardBehavior);
+ }
+
+ void TearDown() override {
+ aura::client::SetScreenPositionClient(root_window(), nullptr);
+ AuraTestBase::TearDown();
+ }
+
+ private:
+ std::unique_ptr<aura::client::ScreenPositionClient> screen_position_client_;
+};
sky 2017/04/28 20:17:51 DISALLOW...
yhanada 2017/05/01 09:12:48 Done.
+
+TEST_F(ImeUtilChromeosTest, RestoreWindowBounds) {
+ const gfx::Rect bounds(10, 20, 100, 200);
+ aura::Window* window =
+ aura::test::CreateTestWindowWithBounds(bounds, root_window());
+
+ EXPECT_EQ(nullptr, window->GetProperty(kVirtualKeyboardRestoreBoundsKey));
+ EXPECT_EQ(bounds, window->bounds());
+
+ RestoreWindowBoundsOnClientFocusLost(window);
+ EXPECT_EQ(bounds, window->bounds());
+
+ gfx::Rect r1(40, 50, 150, 200);
+ window->SetProperty(kVirtualKeyboardRestoreBoundsKey, new gfx::Rect(r1));
+ RestoreWindowBoundsOnClientFocusLost(window);
+ EXPECT_EQ(r1, window->bounds());
+ EXPECT_EQ(nullptr, window->GetProperty(kVirtualKeyboardRestoreBoundsKey));
+}
+
+TEST_F(ImeUtilChromeosTest, EnsureWindowNotInRect_NotCovered) {
+ const gfx::Rect bounds(0, 0, 100, 200);
+ aura::Window* window =
+ aura::test::CreateTestWindowWithBounds(bounds, root_window());
+ EXPECT_EQ(bounds, window->bounds());
+ EXPECT_EQ(bounds, window->GetBoundsInScreen());
+
+ // The rect doesn't overlap on the window.
+ gfx::Rect rect(300, 300, 100, 100);
+ EXPECT_TRUE(gfx::IntersectRects(window->GetBoundsInScreen(), rect).IsEmpty());
+ EnsureWindowNotInRect(window, rect);
+ // The bounds should not be changed.
+ EXPECT_EQ(bounds, window->bounds());
+ EXPECT_EQ(bounds, window->GetBoundsInScreen());
+}
+
+TEST_F(ImeUtilChromeosTest, EnsureWindowNotInRect_MoveUp) {
+ const gfx::Rect original_bounds(10, 100, 100, 10);
+ aura::Window* window =
+ aura::test::CreateTestWindowWithBounds(original_bounds, root_window());
+ EXPECT_EQ(original_bounds, window->bounds());
+ EXPECT_EQ(original_bounds, window->GetBoundsInScreen());
+
+ // The rect overlaps the window. The window is moved up by
+ // EnsureWindowNotInRect.
+ gfx::Rect rect(50, 50, 200, 200);
+ EXPECT_FALSE(
+ gfx::IntersectRects(window->GetBoundsInScreen(), rect).IsEmpty());
+ EnsureWindowNotInRect(window, rect);
+ EXPECT_EQ(gfx::Rect(10, 40, 100, 10), window->bounds());
+ EXPECT_EQ(gfx::Rect(10, 40, 100, 10), window->GetBoundsInScreen());
+}
+
+TEST_F(ImeUtilChromeosTest, EnsureWindowNotInRect_MoveToTop) {
+ const gfx::Rect original_bounds(10, 10, 100, 100);
+ aura::Window* window =
+ aura::test::CreateTestWindowWithBounds(original_bounds, root_window());
+ EXPECT_EQ(original_bounds, window->bounds());
+ EXPECT_EQ(original_bounds, window->GetBoundsInScreen());
+
+ // The rect overlaps the window. The window is moved up by
+ // EnsureWinodwNotInRect, but there is not enough space above the window.
+ gfx::Rect rect(50, 50, 200, 200);
+ EXPECT_FALSE(
+ gfx::IntersectRects(window->GetBoundsInScreen(), rect).IsEmpty());
+ EnsureWindowNotInRect(window, rect);
+ EXPECT_EQ(gfx::Rect(10, 0, 100, 100), window->bounds());
+ EXPECT_EQ(gfx::Rect(10, 0, 100, 100), window->GetBoundsInScreen());
+}
+
+TEST_F(ImeUtilChromeosTest, MoveUpThenRestore) {
+ const gfx::Rect original_bounds(50, 50, 100, 100);
+ aura::Window* window =
+ aura::test::CreateTestWindowWithBounds(original_bounds, root_window());
+ EXPECT_EQ(original_bounds, window->bounds());
+ EXPECT_EQ(original_bounds, window->GetBoundsInScreen());
+
+ // EnsureWindowNotInRect moves up the window.
+ gfx::Rect rect(50, 50, 200, 200);
+ EXPECT_FALSE(
+ gfx::IntersectRects(window->GetBoundsInScreen(), rect).IsEmpty());
+ EnsureWindowNotInRect(window, rect);
+ EXPECT_EQ(gfx::Rect(50, 0, 100, 100), window->bounds());
+ EXPECT_EQ(gfx::Rect(50, 0, 100, 100), window->GetBoundsInScreen());
+
+ // The new rect doesn't overlap the moved window bounds, but still overlaps
+ // the original window bounds.
+ rect = gfx::Rect(50, 120, 200, 200);
+ EXPECT_FALSE(gfx::IntersectRects(rect, original_bounds).IsEmpty());
+ EnsureWindowNotInRect(window, rect);
+ EXPECT_EQ(gfx::Rect(50, 20, 100, 100), window->bounds());
+ EXPECT_EQ(gfx::Rect(50, 20, 100, 100), window->GetBoundsInScreen());
+
+ // Now the rect doesn't overlap the original window bounds. The original
+ // window bounds should be restored.
+ rect = gfx::Rect(200, 200, 200, 200);
+ EXPECT_TRUE(gfx::IntersectRects(rect, original_bounds).IsEmpty());
+ EnsureWindowNotInRect(window, rect);
+ EXPECT_EQ(original_bounds, window->bounds());
+ EXPECT_EQ(original_bounds, window->GetBoundsInScreen());
+}
+
+} // namespace wm
« ui/wm/core/ime_util_chromeos.cc ('K') | « ui/wm/core/ime_util_chromeos.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698