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

Unified Diff: ui/views/controls/tabbed_pane/tabbed_pane.h

Issue 2578303003: a11y: Add a11y information to views::Tab and manually ignore its a11y children. (Closed)
Patch Set: Add tests. Created 3 years, 11 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: ui/views/controls/tabbed_pane/tabbed_pane.h
diff --git a/ui/views/controls/tabbed_pane/tabbed_pane.h b/ui/views/controls/tabbed_pane/tabbed_pane.h
index 4ff3c6922bd1c88733c943f7e2fa833d70e1e662..4b40b1df1f73ab99d1543db74a1e8f85d1d0f35c 100644
--- a/ui/views/controls/tabbed_pane/tabbed_pane.h
+++ b/ui/views/controls/tabbed_pane/tabbed_pane.h
@@ -60,12 +60,11 @@ class VIEWS_EXPORT TabbedPane : public View {
private:
friend class FocusTraversalTest;
friend class Tab;
+ friend class TabbedPaneMacAccessibilityTest;
tapted 2017/01/06 04:25:11 this should have a views::test:: namespace on it.
Patti Lor 2017/01/09 22:39:27 Done.
friend class TabStrip;
FRIEND_TEST_ALL_PREFIXES(TabbedPaneTest, AddAndSelect);
FRIEND_TEST_ALL_PREFIXES(TabbedPaneTest, ArrowKeyBindings);
-
- // Get the Tab (the tabstrip view, not its content) at the valid |index|.
- Tab* GetTabAt(int index);
tapted 2017/01/06 04:25:11 huh. I guess this got removed at some point and th
Patti Lor 2017/01/09 22:39:27 To do this, I also had to move the class declarati
+ FRIEND_TEST_ALL_PREFIXES(TabbedPaneTest, SelectTabWithAccessibleAction);
// Get the Tab (the tabstrip view, not its content) at the selected index.
Tab* GetSelectedTab();
@@ -121,6 +120,8 @@ class Tab : public View {
void OnGestureEvent(ui::GestureEvent* event) override;
gfx::Size GetPreferredSize() const override;
const char* GetClassName() const override;
+ void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
+ bool HandleAccessibleAction(const ui::AXActionData& action_data) override;
void OnFocus() override;
void OnBlur() override;
bool OnKeyPressed(const ui::KeyEvent& event) override;

Powered by Google App Engine
This is Rietveld 408576698