Skip to content

Commit

Permalink
Move ui/base/accessibility to ui/accessibility.
Browse files Browse the repository at this point in the history
Gets rid of the enums defined in accessibility_types.h and switches to the
enums defined in ui/accessibility/ax_enums.h, which are shared by "content".

Eventually, content and views may share more common accessibility code, but
moving to the same enums is a good first step.

BUG=316726
R=piman@chromium.org, sky@chromium.org, tfarina@chromium.org

Review URL: https://codereview.chromium.org/126763003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253207 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
dmazzoni@chromium.org committed Feb 25, 2014
1 parent bf26d22 commit 739c8bc
Show file tree
Hide file tree
Showing 210 changed files with 672 additions and 767 deletions.
8 changes: 4 additions & 4 deletions ash/accelerators/exit_warning_handler.cc
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
#include "base/time/time.h"
#include "base/timer/timer.h"
#include "grit/ash_strings.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/aura/window.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/canvas.h"
Expand Down Expand Up @@ -94,9 +94,9 @@ class ExitWarningWidgetDelegateView : public views::WidgetDelegateView {
views::WidgetDelegateView::OnPaint(canvas);
}

virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE {
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE {
state->name = accessible_name_;
state->role = ui::AccessibilityTypes::ROLE_ALERT;
state->role = ui::AX_ROLE_ALERT;
}

private:
Expand Down Expand Up @@ -195,7 +195,7 @@ void ExitWarningHandler::Show() {
widget_->GetNativeView()->SetName("ExitWarningWindow");
widget_->Show();

delegate->NotifyAccessibilityEvent(ui::AccessibilityTypes::EVENT_ALERT, true);
delegate->NotifyAccessibilityEvent(ui::AX_EVENT_ALERT, true);
}

void ExitWarningHandler::Hide() {
Expand Down
1 change: 1 addition & 0 deletions ash/ash.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -733,6 +733,7 @@
'dependencies': [
'../skia/skia.gyp:skia',
'../testing/gtest.gyp:gtest',
'../ui/accessibility/accessibility.gyp:ax_gen',
'../ui/app_list/app_list.gyp:app_list_test_support',
'../ui/views/views.gyp:views_test_support',
'ash',
Expand Down
6 changes: 3 additions & 3 deletions ash/shelf/alternate_app_list_button.cc
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
#include "ash/shell.h"
#include "grit/ash_resources.h"
#include "grit/ash_strings.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/compositor/layer.h"
Expand Down Expand Up @@ -165,8 +165,8 @@ void AlternateAppListButton::OnPaint(gfx::Canvas* canvas) {
}

void AlternateAppListButton::GetAccessibleState(
ui::AccessibleViewState* state) {
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
ui::AXViewState* state) {
state->role = ui::AX_ROLE_BUTTON;
state->name = host_->GetAccessibleName(this);
}

Expand Down
2 changes: 1 addition & 1 deletion ash/shelf/alternate_app_list_button.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ class AlternateAppListButton : public views::ImageButton {
virtual void OnMouseEntered(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseExited(const ui::MouseEvent& event) OVERRIDE;
virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;

// ui::EventHandler overrides:
virtual void OnGestureEvent(ui::GestureEvent* event) OVERRIDE;
Expand Down
6 changes: 3 additions & 3 deletions ash/shelf/app_list_button.cc
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
#include "ash/shelf/shelf_constants.h"
#include "grit/ash_resources.h"
#include "grit/ash_strings.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/compositor/layer.h"
Expand Down Expand Up @@ -123,8 +123,8 @@ void AppListButton::OnMouseExited(const ui::MouseEvent& event) {
host_->MouseExitedButton(this);
}

void AppListButton::GetAccessibleState(ui::AccessibleViewState* state) {
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
void AppListButton::GetAccessibleState(ui::AXViewState* state) {
state->role = ui::AX_ROLE_BUTTON;
state->name = host_->GetAccessibleName(this);
}

Expand Down
2 changes: 1 addition & 1 deletion ash/shelf/app_list_button.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class AppListButton : public views::ImageButton {
virtual void OnMouseMoved(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseEntered(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseExited(const ui::MouseEvent& event) OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;

private:
ShelfButtonHost* host_;
Expand Down
6 changes: 3 additions & 3 deletions ash/shelf/shelf_button.cc
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
#include "ash/shelf/shelf_layout_manager.h"
#include "grit/ash_resources.h"
#include "skia/ext/image_operations.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/compositor/layer.h"
#include "ui/compositor/scoped_layer_animation_settings.h"
Expand Down Expand Up @@ -392,8 +392,8 @@ void ShelfButton::OnMouseExited(const ui::MouseEvent& event) {
host_->MouseExitedButton(this);
}

void ShelfButton::GetAccessibleState(ui::AccessibleViewState* state) {
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
void ShelfButton::GetAccessibleState(ui::AXViewState* state) {
state->role = ui::AX_ROLE_BUTTON;
state->name = host_->GetAccessibleName(this);
}

Expand Down
2 changes: 1 addition & 1 deletion ash/shelf/shelf_button.h
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ class ASH_EXPORT ShelfButton : public views::CustomButton {
virtual void OnMouseMoved(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseEntered(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseExited(const ui::MouseEvent& event) OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;
virtual void Layout() OVERRIDE;
virtual void ChildPreferredSizeChanged(views::View* child) OVERRIDE;
virtual void OnFocus() OVERRIDE;
Expand Down
6 changes: 3 additions & 3 deletions ash/shelf/shelf_view.cc
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,10 @@
#include "base/metrics/histogram.h"
#include "grit/ash_resources.h"
#include "grit/ash_strings.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/aura/client/screen_position_client.h"
#include "ui/aura/window.h"
#include "ui/aura/window_event_dispatcher.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/models/simple_menu_model.h"
#include "ui/base/resource/resource_bundle.h"
Expand Down Expand Up @@ -1529,8 +1529,8 @@ views::FocusTraversable* ShelfView::GetPaneFocusTraversable() {
return this;
}

void ShelfView::GetAccessibleState(ui::AccessibleViewState* state) {
state->role = ui::AccessibilityTypes::ROLE_TOOLBAR;
void ShelfView::GetAccessibleState(ui::AXViewState* state) {
state->role = ui::AX_ROLE_TOOLBAR;
state->name = l10n_util::GetStringUTF16(IDS_ASH_SHELF_ACCESSIBLE_NAME);
}

Expand Down
2 changes: 1 addition & 1 deletion ash/shelf/shelf_view.h
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,7 @@ class ASH_EXPORT ShelfView : public views::View,
virtual gfx::Size GetPreferredSize() OVERRIDE;
virtual void OnBoundsChanged(const gfx::Rect& previous_bounds) OVERRIDE;
virtual FocusTraversable* GetPaneFocusTraversable() OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;

// Overridden from ui::EventHandler:
virtual void OnGestureEvent(ui::GestureEvent* event) OVERRIDE;
Expand Down
8 changes: 4 additions & 4 deletions ash/system/chromeos/network/tray_network.cc
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
#include "grit/ash_resources.h"
#include "grit/ash_strings.h"
#include "third_party/cros_system_api/dbus/service_constants.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/views/controls/image_view.h"
Expand Down Expand Up @@ -101,9 +101,9 @@ class NetworkTrayView : public TrayItemView,
}

// views::View override.
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE {
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE {
state->name = connection_status_string_;
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
state->role = ui::AX_ROLE_BUTTON;
}

// network_icon::AnimationObserver
Expand All @@ -123,7 +123,7 @@ class NetworkTrayView : public TrayItemView,
if (new_connection_status_string != connection_status_string_) {
connection_status_string_ = new_connection_status_string;
if(!connection_status_string_.empty())
NotifyAccessibilityEvent(ui::AccessibilityTypes::EVENT_ALERT, true);
NotifyAccessibilityEvent(ui::AX_EVENT_ALERT, true);
}
}

Expand Down
10 changes: 5 additions & 5 deletions ash/system/chromeos/power/tray_power.cc
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
#include "grit/ash_strings.h"
#include "third_party/icu/source/i18n/unicode/fieldpos.h"
#include "third_party/icu/source/i18n/unicode/fmtable.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/message_center/message_center.h"
#include "ui/message_center/notification.h"
Expand Down Expand Up @@ -61,9 +61,9 @@ class PowerTrayView : public views::ImageView {
}

// Overriden from views::View.
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE {
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE {
state->name = accessible_name_;
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
state->role = ui::AX_ROLE_BUTTON;
}

void UpdateStatus(bool battery_alert) {
Expand All @@ -72,7 +72,7 @@ class PowerTrayView : public views::ImageView {

if (battery_alert) {
accessible_name_ = PowerStatus::Get()->GetAccessibleNameString();
NotifyAccessibilityEvent(ui::AccessibilityTypes::EVENT_ALERT, true);
NotifyAccessibilityEvent(ui::AX_EVENT_ALERT, true);
}
}

Expand Down Expand Up @@ -103,7 +103,7 @@ class PowerTrayView : public views::ImageView {

accessible_name_ = ui::ResourceBundle::GetSharedInstance().
GetLocalizedString(IDS_CHARGER_REPLACEMENT_ACCESSIBILTY_NOTIFICATION);
NotifyAccessibilityEvent(ui::AccessibilityTypes::EVENT_ALERT, true);
NotifyAccessibilityEvent(ui::AX_EVENT_ALERT, true);
++spring_charger_spoken_notification_count_;

if (spring_charger_spoken_notification_count_ == 1) {
Expand Down
1 change: 0 additions & 1 deletion ash/system/chromeos/settings/tray_settings.cc
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
#include "grit/ash_resources.h"
#include "grit/ash_strings.h"
#include "third_party/skia/include/core/SkColor.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/image/image.h"
#include "ui/views/controls/image_view.h"
Expand Down
6 changes: 3 additions & 3 deletions ash/system/tray/actionable_view.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
#include "ash/system/tray/actionable_view.h"

#include "ash/ash_constants.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/gfx/canvas.h"

namespace ash {
Expand Down Expand Up @@ -63,8 +63,8 @@ void ActionableView::SetAccessibleName(const base::string16& name) {
accessible_name_ = name;
}

void ActionableView::GetAccessibleState(ui::AccessibleViewState* state) {
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
void ActionableView::GetAccessibleState(ui::AXViewState* state) {
state->role = ui::AX_ROLE_BUTTON;
state->name = accessible_name_;
}

Expand Down
2 changes: 1 addition & 1 deletion ash/system/tray/actionable_view.h
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ class ASH_EXPORT ActionableView : public views::View {
virtual bool OnMousePressed(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseReleased(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseCaptureLost() OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;
virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
virtual void OnFocus() OVERRIDE;
virtual void OnBlur() OVERRIDE;
Expand Down
8 changes: 4 additions & 4 deletions ash/system/tray/hover_highlight_view.cc
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include "ash/system/tray/tray_constants.h"
#include "ash/system/tray/view_click_listener.h"
#include "grit/ui_resources.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/canvas.h"
#include "ui/gfx/font_list.h"
Expand Down Expand Up @@ -137,12 +137,12 @@ bool HoverHighlightView::PerformAction(const ui::Event& event) {
return true;
}

void HoverHighlightView::GetAccessibleState(ui::AccessibleViewState* state) {
void HoverHighlightView::GetAccessibleState(ui::AXViewState* state) {
ActionableView::GetAccessibleState(state);

if (checkable_) {
state->role = ui::AccessibilityTypes::ROLE_CHECKBUTTON;
state->state = checked_ ? ui::AccessibilityTypes::STATE_CHECKED : 0;
state->role = ui::AX_ROLE_CHECK_BOX;
state->state = checked_ ? ui::AX_STATE_CHECKED : 0;
}
}

Expand Down
2 changes: 1 addition & 1 deletion ash/system/tray/hover_highlight_view.h
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ class HoverHighlightView : public ActionableView {
virtual bool PerformAction(const ui::Event& event) OVERRIDE;

// Overridden from views::View.
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;
virtual gfx::Size GetPreferredSize() OVERRIDE;
virtual int GetHeightForWidth(int width) OVERRIDE;
virtual void OnMouseEntered(const ui::MouseEvent& event) OVERRIDE;
Expand Down
2 changes: 1 addition & 1 deletion ash/system/tray/system_tray_bubble.cc
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ void SystemTrayBubble::InitView(views::View* anchor,

if (bubble_view_->CanActivate()) {
bubble_view_->NotifyAccessibilityEvent(
ui::AccessibilityTypes::EVENT_ALERT, true);
ui::AX_EVENT_ALERT, true);
}
}

Expand Down
6 changes: 3 additions & 3 deletions ash/system/tray/tray_background_view.cc
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@
#include "ash/system/tray/tray_event_filter.h"
#include "ash/wm/window_animations.h"
#include "grit/ash_resources.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/aura/window.h"
#include "ui/aura/window_event_dispatcher.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/canvas.h"
#include "ui/gfx/image/image_skia.h"
Expand Down Expand Up @@ -377,8 +377,8 @@ void TrayBackgroundView::ChildPreferredSizeChanged(views::View* child) {
PreferredSizeChanged();
}

void TrayBackgroundView::GetAccessibleState(ui::AccessibleViewState* state) {
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
void TrayBackgroundView::GetAccessibleState(ui::AXViewState* state) {
state->role = ui::AX_ROLE_BUTTON;
state->name = GetAccessibleNameForTray();
}

Expand Down
2 changes: 1 addition & 1 deletion ash/system/tray/tray_background_view.h
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ class ASH_EXPORT TrayBackgroundView : public ActionableView,
virtual void OnMouseEntered(const ui::MouseEvent& event) OVERRIDE;
virtual void OnMouseExited(const ui::MouseEvent& event) OVERRIDE;
virtual void ChildPreferredSizeChanged(views::View* child) OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;
virtual void AboutToRequestFocusFromTabTraversal(bool reverse) OVERRIDE;

// Overridden from internal::ActionableView.
Expand Down
6 changes: 3 additions & 3 deletions ash/system/tray/tray_item_more.cc
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include "ash/system/tray/system_tray_item.h"
#include "ash/system/tray/tray_constants.h"
#include "grit/ash_resources.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/image/image.h"
#include "ui/views/controls/image_view.h"
Expand Down Expand Up @@ -98,8 +98,8 @@ void TrayItemMore::Layout() {
}
}

void TrayItemMore::GetAccessibleState(ui::AccessibleViewState* state) {
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
void TrayItemMore::GetAccessibleState(ui::AXViewState* state) {
state->role = ui::AX_ROLE_BUTTON;
state->name = accessible_name_;
}

Expand Down
2 changes: 1 addition & 1 deletion ash/system/tray/tray_item_more.h
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ class TrayItemMore : public ActionableView {

// Overridden from views::View.
virtual void Layout() OVERRIDE;
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE;

SystemTrayItem* owner_;
// True if |more_| should be shown.
Expand Down
6 changes: 3 additions & 3 deletions ash/system/tray_caps_lock.cc
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
#include "ash/system/tray/tray_constants.h"
#include "grit/ash_resources.h"
#include "grit/ash_strings.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/image/image.h"
#include "ui/views/controls/image_view.h"
Expand Down Expand Up @@ -88,8 +88,8 @@ class CapsLockDefaultView : public ActionableView {
text_size.height()));
}

virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE {
state->role = ui::AccessibilityTypes::ROLE_PUSHBUTTON;
virtual void GetAccessibleState(ui::AXViewState* state) OVERRIDE {
state->role = ui::AX_ROLE_BUTTON;
state->name = text_label_->text();
}

Expand Down
Loading

0 comments on commit 739c8bc

Please sign in to comment.