diff options
| author | mhm@chromium.org <mhm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-09 04:00:49 +0000 |
|---|---|---|
| committer | mhm@chromium.org <mhm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-09 04:00:49 +0000 |
| commit | 70d38b09d6f504b6a47b4d0e8f0a9b2f5d615373 (patch) | |
| tree | efcc27929e40b9e71d0284d8094dab970945214d /views/controls/progress_bar.cc | |
| parent | 7f265f9db6a502cb8909636d350afd0dbf6c5539 (diff) | |
| download | chromium_src-70d38b09d6f504b6a47b4d0e8f0a9b2f5d615373.zip chromium_src-70d38b09d6f504b6a47b4d0e8f0a9b2f5d615373.tar.gz chromium_src-70d38b09d6f504b6a47b4d0e8f0a9b2f5d615373.tar.bz2 | |
[accessibility] Allow views to have multiple accessibility states and add correct roles.
Allow views to contain multiple accessibility states since states are not mutually exclusive, we treat them as an uint32. Add missing roles to couple of views.
BUG=None
TEST=Run unit tests, and check accessibility tree.
Review URL: http://codereview.chromium.org/1527019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44054 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/controls/progress_bar.cc')
| -rw-r--r-- | views/controls/progress_bar.cc | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/views/controls/progress_bar.cc b/views/controls/progress_bar.cc index de9cd59..19f10c5 100644 --- a/views/controls/progress_bar.cc +++ b/views/controls/progress_bar.cc @@ -187,16 +187,12 @@ void ProgressBar::SetEnabled(bool enabled) { bool ProgressBar::GetAccessibleRole(AccessibilityTypes::Role* role) { DCHECK(role); - if (role == NULL) - return false; - *role = AccessibilityTypes::ROLE_TEXT; + *role = AccessibilityTypes::ROLE_PROGRESSBAR; return true; } bool ProgressBar::GetAccessibleState(AccessibilityTypes::State* state) { DCHECK(state); - if (state == NULL) - return false; *state = AccessibilityTypes::STATE_READONLY; return true; } |