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

Unified Diff: ui/views/accessibility/native_view_accessibility_unittest.cc

Issue 2795843002: Move implementation of accessibility actions to views::View (Closed)
Patch Set: Fix win compile Created 3 years, 9 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 | « ui/views/accessibility/native_view_accessibility_base.cc ('k') | ui/views/view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/accessibility/native_view_accessibility_unittest.cc
diff --git a/ui/views/accessibility/native_view_accessibility_unittest.cc b/ui/views/accessibility/native_view_accessibility_unittest.cc
index 92eb0035e57b2067f1415f0c2799fa9a4bc4b843..f57d7a217366ff3d457a95ded17d321c27374b05 100644
--- a/ui/views/accessibility/native_view_accessibility_unittest.cc
+++ b/ui/views/accessibility/native_view_accessibility_unittest.cc
@@ -70,6 +70,13 @@ class NativeViewAccessibilityTest : public ViewsTestBase {
label_accessibility_.get());
}
+ bool SetFocused(NativeViewAccessibilityBase* view_accessibility,
+ bool focused) {
+ ui::AXActionData data;
+ data.action = focused ? ui::AX_ACTION_FOCUS : ui::AX_ACTION_BLUR;
+ return view_accessibility->AccessibilityPerformAction(data);
+ }
+
protected:
views::Widget* widget_;
TestButton* button_;
@@ -117,17 +124,17 @@ TEST_F(NativeViewAccessibilityTest, WritableFocus) {
button_->SetFocusBehavior(View::FocusBehavior::ALWAYS);
EXPECT_EQ(nullptr, button_->GetFocusManager()->GetFocusedView());
EXPECT_EQ(nullptr, button_accessibility()->GetFocus());
- EXPECT_TRUE(button_accessibility()->SetFocused(true));
+ EXPECT_TRUE(SetFocused(button_accessibility(), true));
EXPECT_EQ(button_, button_->GetFocusManager()->GetFocusedView());
EXPECT_EQ(button_->GetNativeViewAccessible(),
button_accessibility()->GetFocus());
- EXPECT_TRUE(button_accessibility()->SetFocused(false));
+ EXPECT_TRUE(SetFocused(button_accessibility(), false));
EXPECT_EQ(nullptr, button_->GetFocusManager()->GetFocusedView());
EXPECT_EQ(nullptr, button_accessibility()->GetFocus());
// If not focusable at all, SetFocused() should return false.
button_->SetEnabled(false);
- EXPECT_FALSE(button_accessibility()->SetFocused(true));
+ EXPECT_FALSE(SetFocused(button_accessibility(), true));
}
// Subclass of NativeViewAccessibility that destroys itself when its
« no previous file with comments | « ui/views/accessibility/native_view_accessibility_base.cc ('k') | ui/views/view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698