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

Unified Diff: ash/wm/overview/transparent_activate_window_button.cc

Issue 251103005: Added arrow key navigation to Overview Mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added tween to the selection widget animation Created 6 years, 6 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: ash/wm/overview/transparent_activate_window_button.cc
diff --git a/ash/wm/overview/transparent_activate_window_button.cc b/ash/wm/overview/transparent_activate_window_button.cc
index 216f0f33c326bc15ae4dd122861b7d89b9da3e07..a92e8bbf7d26271c90579ad418622ded36e931d3 100644
--- a/ash/wm/overview/transparent_activate_window_button.cc
+++ b/ash/wm/overview/transparent_activate_window_button.cc
@@ -53,7 +53,8 @@ TransparentActivateWindowButton::TransparentActivateWindowButton(
aura::Window* activate_window)
: event_handler_widget_(InitEventHandler(activate_window->GetRootWindow())),
activate_window_(activate_window) {
- views::View* transparent_button = new TransparentButton(this);
+ views::Button* transparent_button = new TransparentButton(this);
+ transparent_button->SetAccessibleName(activate_window->title());
event_handler_widget_->SetContentsView(transparent_button);
}
@@ -61,6 +62,11 @@ void TransparentActivateWindowButton::SetBounds(const gfx::Rect& bounds) {
event_handler_widget_->SetBounds(bounds);
}
+void TransparentActivateWindowButton::SendFocusAlert() const {
+ event_handler_widget_->GetContentsView()->
flackr 2014/06/05 20:16:33 // TODO(nsatragno): Actually focus the button. :-)
Nina 2014/06/05 22:06:23 According to what we discussed off-patch, this doe
+ NotifyAccessibilityEvent(ui::AX_EVENT_FOCUS, true);
+}
+
TransparentActivateWindowButton::~TransparentActivateWindowButton() {
}

Powered by Google App Engine
This is Rietveld 408576698